asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Steven Jacobs (Code Review)" <>
Subject Change in asterixdb[master]: ASTERIXDB-1747 Implemented full lifecycle capabilities for d...
Date Mon, 06 Feb 2017 22:32:08 GMT
Steven Jacobs has posted comments on this change.

Change subject: ASTERIXDB-1747 Implemented full lifecycle capabilities for distributed jobs

Patch Set 10:


Addressed comments. As far as tests go, the functionality is tested in BAD, which is run against
all changes to Asterix, which should prevent regression.
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/

PS7, Line 109: else
> formatting?
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/

PS10, Line 359: removeActivityClusterGraphConstraints
> This seems to be used. Do we leak constraints here?
Good catch! FIXED!
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/executor/

PS10, Line 495: JavaSerializationUtils
> Do we serialize this for every TaskAttempt? Why do we need to do that?
Previously this was serialized for every invocation of StartTasks. Now it is only serialized
if the job is not predistributed. I also added the check for changed, since I noticed that
it's not actually used unless changed is true.
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/job/

PS10, Line 67: ActivityClusterGraph
> Why is this not final anymore?
Made it final

PS10, Line 69: scheduler
> Why is this not final anymore?
This is created within the two public constructors, which call the private constructor. Since
it isn't assigned in the private constructor, it can't be final.
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/work/

PS10, Line 49: removeJobSpecification
> constraints are not removed

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I59c3422d5c1ab7756a6a4685ac527dfe50434954
Gerrit-PatchSet: 10
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Steven Jacobs <>
Gerrit-Reviewer: Ian Maxon <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Steven Jacobs <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Xikui Wang <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message