Murtadha Hubail has posted comments on this change.

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


Patch Set 3:

@Till and @Yingyi,
I removed the API using resource id and the checks for persistent local 
resources. Please have a look at the change now. If you are happy with it, I 
will submit the Asterix side based on this change. Otherwise, I will revert 
these changes and return the checks.

Note that the deprecated methods on ILocalResourceRepository can be removed 
since no one is using them (at least in hyracks and Asterix).

-- 
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: 3
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