dolthub / dolt

Dolt – Git for Data
Apache License 2.0
17.36k stars 488 forks source link

Optimize JSON_REMOVE on `IndexedJsonDocument` #8103

Open nicktobey opened 4 days ago

nicktobey commented 4 days ago

This PR includes a new implementation of the JSON_REMOVE function that leverages the new indexed JSON storage format.

For JSON documents that span multiple chunks, only the affected chunks need to be loaded and modified, allowing operations to scale with the size of the removed value instead of the size of the entire document.

coffeegoddd commented 4 days ago

@nicktobey DOLT

comparing_percentages
100.000000 to 100.000000
version result total
56eaca2 ok 5937457
version total_tests
56eaca2 5937457
correctness_percentage
100.0