Page MenuHomePhabricator

Don't add the TID by default in storage module
Closed, ResolvedPublic

Description

Currently if there's no tid index in the table, the storage module adds an internal _tid parameter. We need to stop doing that, because for the indexing tables in the new storage design we don't need the TID as an index, we index by timestamp.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 17 2017, 11:02 PM

Right now we can get away from this by using the timeuuid as an index in the timeline tables as we can create TIDs for arbitrary dates. However, the plan for the key_value buckets was to start simply overwriting the content, so this automatic generation of a tid will get in our way..

Pchelolo closed this task as Resolved.Jul 21 2017, 5:49 PM

Merged to the next-gem branch. Resolving.