Steven Jacobs has posted comments on this change. Change subject: Prepare AsterixDB for Pre-Distributed Jobs ......................................................................
Patch Set 3: (3 comments) I made the two changes. I will submit the new patchset now. Can you start looking at the BAD change as well? The problem here is that I've started moving forward with new Channels stuff, since the Nonpure review was taking long, so we need to get a checkpoint build-able BAD branch in place. This way we can: 1) Enable Ian to add the BAD check on Asterix changes 2) Have the master branch build-able for BAD from now on https://asterix-gerrit.ics.uci.edu/#/c/1366/3/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/client/HyracksClientInterfaceFunctions.java File hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/client/HyracksClientInterfaceFunctions.java: Line 112: private StartJobFunction(DeploymentId deploymentId, byte[] acggfBytes, EnumSet<JobFlag> jobFlags, JobId jobId) { > why is this private? this is the only Constructor that can be used to set a I'll make it public for now. https://asterix-gerrit.ics.uci.edu/#/c/1366/3/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NodeControllerService.java: Line 363: public Map<JobId, ActivityClusterGraph> getACGMap() { > should we rename this to getActivityClusterGraphMap()?. just a suggestion. Done https://asterix-gerrit.ics.uci.edu/#/c/1366/3/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java: Line 197: acgMap.put(jobId, acg); > this map is only growing. when/how do we remove activity cluster graphs fro This is exactly the plan :) The problem here is that we need to get the master BAD branch to a build-able state, so this Asterix change is incremental. There will be lots more changes to come -- To view, visit https://asterix-gerrit.ics.uci.edu/1366 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id809f4b563bbed808c7764d1af664a15919db35b Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Steven Jacobs <sjaco...@ucr.edu> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Steven Jacobs <sjaco...@ucr.edu> Gerrit-Reviewer: Till Westmann <ti...@apache.org> Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com> Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com> Gerrit-HasComments: Yes