Murtadha Hubail has posted comments on this change.

Change subject: Add support for persistent local resources to 
IndexLifecycleManager
......................................................................


Patch Set 2:

Yes, the goal is correct, and you dug correctly. In the logs, we have the 
dataset id and the resource id. We don't store the string.

During recovery (REDO only), we load all resources and construct the mapping 
while loading them.

During aborts (UNDO), the indexes are guaranteed to be open, so we fetch them 
using the dataset id and the resource id. This mapping is maintained in 
DatasetLifeCycleManager for the opened datasets only, and removed once they are 
closed.

Therefore, strings are enough for the API between IndexDataFlowHelper and 
IndexLifeCycleManager.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2e8e974fc2f746959639ce94351f8e419a7f9093
Gerrit-PatchSet: 2
Gerrit-Project: hyracks
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-Reviewer: Young-Seok Kim <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: No

Reply via email to