database icon indicating copy to clipboard operation
database copied to clipboard

Primary key permission

Open fogelito opened this issue 2 years ago • 5 comments

We would like to changes in all _perms collection from using Varchar DocumentId => integer document id For better performance and shrinking the index resources. Since this is internal it would be relatively easier changes.

fogelito avatar Jul 04 '23 13:07 fogelito

@fogelito What's the benefit of this change? Please add a description to the PR :pray:

abnegate avatar Aug 01 '23 00:08 abnegate

We have a Benchmark for this on Asana

fogelito avatar Aug 01 '23 07:08 fogelito

@fogelito Awesome let's add a migration PR in Appwrite for this as well

abnegate avatar Aug 02 '23 23:08 abnegate

Still need to make changes here as in the update for example we need to pass the internal id after read the original doc. Need to think the affect on relations as well. Currently original values saved as well.

fogelito avatar Aug 09 '23 10:08 fogelito

@fogelito Cool will mark this as draft for now 👌

abnegate avatar Aug 10 '23 22:08 abnegate