asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <>
Subject Change in asterixdb[master]: Supports flexible degree of parallelism.
Date Wed, 21 Dec 2016 21:08:07 GMT
Till Westmann has posted comments on this change.

Change subject: Supports flexible degree of parallelism.

Patch Set 11:

Commit Message:

Line 9: Detailed list of changes include:
Really appreciate this list!
File asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/

Line 358:     // Computes the location constraints based on user-configured parallism parameter.

Line 366:             int parallelism = parallelismHint <= 0 ? -2 * ncMap.size() : parallelismHint;
What's the meaning of -2 here?
File asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/

Line 169: import org.apache.asterix.translator.TypeTranslator;
There's something funny going on with the imports ... they are different for me ..
File asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/

Line 91:         return testArgs;
If there's another patchset we could just replace this method body with

    return LangExecutionUtil.buildTestsInXml(TestCaseContext.DEFAULT_REPEATED_TESTSUITE_XML_NAME);

and remove the static import - but don't create a new patchset for this.
File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/

Line 130:                         null);
It seems that there could be some utility method the could determine the StructuralPropertiesVector
based on the inputs, the context, and the classes of the partitioning properties that would
make this more intuitive.
But I'm not sure (and I'm not sure where to put it) - it's just a thought.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: If70faf52bed995c7098cc343f2fabd1b6c8d96e7
Gerrit-PatchSet: 11
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Michael Blow <>
Gerrit-Reviewer: Steven Jacobs <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message