Raise exception if a view to save's id already exists

[nvoxland-al]

🚀 🚀 Pull Request

Impact

• Bug fix (non-breaking change which fixes expected existing functionality)
• Enhancement/New feature (adds functionality without impacting existing logic)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

A script like this unexpectedly overwrites the existing view:

ds = deeplake.empty("debug_branch_view", overwrite=True)
ds.create_tensor("test")
ds.extend({"test": np.random.rand(100, 100, 3)})
ds.commit("first_commit")
view_1 = ds[0:10]
view_1.save_view(id = "view_id")

view_2 = ds[20:30]
view_2.save_view(id = "view_id")

print(ds.get_views())
print(len(ds.get_views()))

This PR change the 2nd save_view to raise an exception

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud