Ildar Absalyamov has posted comments on this change.

Change subject: [ASTERIXDB-2090][STO] Fix componentIds for bulk load
......................................................................


Patch Set 2:

General question - what is the effect of having invalid LSMDiskComponentId 
right now? Does merging components with bulkloaded one have the same semantics? 
Or is searching the bulkloaded components somehow affected?  Does not seem that 
any of the tests breaks\requires modification. If it's just the matter of 
WARNING message in the log I would say it's better to implement timestamp-based 
componentIDs even if that requires more time since the problem does not require 
immediate solution.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2015
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d85c1a0a2e9f04bac1c56113987aa48e1beafe2
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ildar Absalyamov <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: No

Reply via email to