-
Type: Task
-
Resolution: Won't Do
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
-
Storage Execution
An engineer unfamiliar with timeseries code, but familiar with the idea that timeseries performs inserts as updates to bucket documents under the hood, can find the timeseries function names confusing. For example:
performTimeseriesWrites makes it sound like a function that can handle any type of operation. But it seems like it just handles inserts.
attemptInsertIntoBucket seems to handle inserts from the name, but is called from insertIntoBucketCatalogForUpdate, so maybe it handles updates.
Then we have performAtomicWrites which seems to be called for updates / deletes, rather than for inserts, which is how the first function uses the term "writes".
Though only three functions were named here, we should clean up other functions as well as a part of this ticket.