Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Mehnaz Tabassum Mahin :
Mehnaz Tabassum Mahin has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17759 )
Change subject: PLEASE EDIT to provide a meaningful commit message!
..
PLEASE EDIT to provide a meaningful commit message!
The following commits from your working branch will be included:
[NO ISSUE][COMP] Add Distinct Cardinality Estimator from samples
Details:
- Add sampling queries for DISTINCT and GROUPBY operators
- Create DISTINCT operators to submit sampling queries for each dataset of
JOIN queries
- Add the Method-of-Moments (MMO) estimator for distinct values from samples
- The estimator can estimate number of distinct values in any attributes of a
dataset, i.e.,
in the attributes given in DISTINCT or GROUPBY operators.
Change-Id: I9f90bddc0fb1b19923a4ceac7436504d8d264689
# Do not delete the following line!
Change-Id: I7443ac65d53aaa3d3579baf3874e660bf562e925
---
A
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/CBODistinctOperatorUtils.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
A
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/DistinctCardinalityEstimation.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java
4 files changed, 598 insertions(+), 0 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/59/17759/1
diff --git
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/CBODistinctOperatorUtils.java
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/CBODistinctOperatorUtils.java
new file mode 100644
index 000..3a9e725
--- /dev/null
+++
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/CBODistinctOperatorUtils.java
@@ -0,0 +1,283 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.asterix.optimizer.rules.cbo;
+
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+
+import org.apache.asterix.om.base.AString;
+import org.apache.asterix.om.constants.AsterixConstantValue;
+import org.apache.asterix.om.functions.BuiltinFunctions;
+import org.apache.commons.lang3.mutable.Mutable;
+import org.apache.commons.lang3.mutable.MutableObject;
+import org.apache.hyracks.algebricks.common.utils.Pair;
+import org.apache.hyracks.algebricks.core.algebra.base.ILogicalExpression;
+import org.apache.hyracks.algebricks.core.algebra.base.ILogicalOperator;
+import org.apache.hyracks.algebricks.core.algebra.base.IOptimizationContext;
+import org.apache.hyracks.algebricks.core.algebra.base.LogicalExpressionTag;
+import org.apache.hyracks.algebricks.core.algebra.base.LogicalOperatorTag;
+import org.apache.hyracks.algebricks.core.algebra.base.LogicalVariable;
+import
org.apache.hyracks.algebricks.core.algebra.expressions.AbstractFunctionCallExpression;
+import
org.apache.hyracks.algebricks.core.algebra.expressions.ConstantExpression;
+import
org.apache.hyracks.algebricks.core.algebra.expressions.ScalarFunctionCallExpression;
+import
org.apache.hyracks.algebricks.core.algebra.expressions.VariableReferenceExpression;
+import
org.apache.hyracks.algebricks.core.algebra.operators.logical.AssignOperator;
+import
org.apache.hyracks.algebricks.core.algebra.operators.logical.DataSourceScanOperator;
+import
org.apache.hyracks.algebricks.core.algebra.operators.logical.DistinctOperator;
+import
org.apache.hyracks.algebricks.core.algebra.operators.logical.GroupByOperator;
+import org.apache.hyracks.api.exceptions.SourceLocation;
+
+public class CBODistinctOperatorUtils {
+
+public static boolean
containsAllGroupByDistinctVarsInScanOp(DataSourceScanOperator scanOp,
+ILogicalOperator grpByDistinctOp) {
+LogicalOperatorTag tag = grpByDistinctOp.getOperatorTag();
+if (tag == LogicalOperatorTag.GROUP || tag ==
LogicalOperatorTag.DISTINCT) {
+List distinctVars =
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From : murali4...@gmail.com has abandoned this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned not needed anymore -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 5 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-MessageType: abandon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Wail Alkowaileet : Attention is currently required from: Ali Alsuliman, Vijay Sarathy, Till Westmann. Wail Alkowaileet has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 ) Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. Patch Set 8: (9 comments) Commit Message: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/d9c47748_2aa82c1b PS8, Line 7: LEASE EDIT to provide a meaningful commit message! : Add cost/card to explain string plan Let's reference an issue for this patch and conform to the standard message. See an example here: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17318 File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/AbstractLogicalOperatorPrettyPrintVisitor.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/64172d09_7108a4d3 PS8, Line 40: protected Double planCard = 0.0; : protected Double planCost = 0.0; : protected Double opCard = 0.0; : protected Double opLocalCost = 0.0; : protected Double opTotalCost = 0.0; Remove and declare them as local variables. They don't need to be members. https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/79a1c312_9abfb1c0 PS8, Line 104: opTotalCost = (Double) getAnnotationValue(op, OperatorAnnotations.OP_COST_TOTAL); You can inline with the declaration: Double opTotalCost =(Double) getAnnotationValue(op, OperatorAnnotations.OP_COST_TOTAL) https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/38781c4c_f527b552 PS8, Line 109: Object getAnnotationValue Let's make this returns a "Double" and rename the method to getAnnotationDoubleValue .. This would protect (in a usability sense) against using it for values other than Doubles :) File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitor.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/da002b96_8dbfc56d PS8, Line 130: planCard Declare it locally and avoid using the member ones (as we should remove them) https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/750f1b8f_b83276bb PS8, Line 132: toLowerCase() Both are already in lowercase. You can remove the call toLowerCase() https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/ec616ef2_5450cded PS8, Line 135: toLowerCase same as above https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/958a1710_db77a16a PS8, Line 150: OperatorAnnotations.OP_COST_LOCAL We can have something similar to protected static final String CARDINALITY = "cardinality"; to OperatorAnnotations.OP_COST_TOTAL, OperatorAnnotations.OP_COST_LOCAL, etc... So we can be consistent File hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/a7b182d4_5f567c7e PS8, Line 279: OperatorAnnotations.OP_COST_LOCAL Same as the STRING plan... we should use the declared ones. -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 Gerrit-Change-Number: 17313 Gerrit-PatchSet: 8 Gerrit-Owner: Vijay Sarathy Gerrit-Reviewer: Ali Alsuliman Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Vijay Sarathy Gerrit-Reviewer: Wail Alkowaileet Gerrit-Attention: Ali Alsuliman Gerrit-Attention: Vijay Sarathy Gerrit-Attention: Till Westmann Gerrit-Comment-Date: Tue, 24 Jan 2023 00:26:21 + Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Vijay Sarathy :
Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Till
Westmann.
Vijay Sarathy has posted comments on this change. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 )
Change subject: PLEASE EDIT to provide a meaningful commit message! Add
cost/card to explain string plan
..
Patch Set 8:
(8 comments)
Patchset:
PS8:
Please review again.
File
asterixdb/asterix-app/src/test/resources/runtimets/results/explain/explain_field_access_closed/explain_field_access_closed.1.plan:
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/0d398680_daf98e81
PS4, Line 39: 51
> Hmm.. […]
Not sure, this change has no relationship to this.
File
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/array-access-pushdown/array-access-pushdown.15.plan:
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/ec202f34_d622cc80
PS4, Line 18: aggregate [$$42] <- [empty-stream()]
> We should also include the costs in a subplan.
Done.
File
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.13.plan:
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/1460bc79_d888e3c8
PS4, Line 1: cardinality: 100.0
> Maybe not related to this change. […]
This is because for queries that do not go through CBO (for e.g. outer joins or
single table queries), no cardinalities are available. Note that single table
queries will be supported very soon, but the change is not merged in yet.
File
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/AbstractLogicalOperatorPrettyPrintVisitor.java:
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/ccd92aa6_4cdad97a
PS4, Line 61:for (Map.Entry anno :
op.getAnnotations().entrySet()) {
: if (anno.getValue() != null &&
anno.getKey().equals(OperatorAnnotations.OP_OUTPUT_CARDINALITY)) {
: cardinality = (double) anno.getValue();
: } else if (anno.getValue() != null &&
anno.getKey().equals(OperatorAnnotations.OP_COST_TOTAL)) {
: cost = (double) anno.getValue();
: }
: }
> We do not need to loop through the annotations. We can get the values for
> both OperatorAnnotations. […]
Done.
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/fd0b99ba_51f1ad80
PS4, Line 77: HashMap
> We probably can break down this method and the one above. […]
Done.
File
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitor.java:
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/d5d4b1f3_45a9ceda
PS4, Line 129: +
> Let's avoid string concatenation. For example: […]
Done.
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313/comment/e4ce8294_8c977fca
PS4, Line 137: cardinality
> Let's declare those constants as static final members in
> AbstractLogicalOperatorPrettyPrintVisitor s […]
Done.
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8
Gerrit-Change-Number: 17313
Gerrit-PatchSet: 8
Gerrit-Owner: Vijay Sarathy
Gerrit-Reviewer: Ali Alsuliman
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Till Westmann
Gerrit-Reviewer: Vijay Sarathy
Gerrit-Reviewer: Wail Alkowaileet
Gerrit-Attention: Wail Alkowaileet
Gerrit-Attention: Ali Alsuliman
Gerrit-Attention: Till Westmann
Gerrit-Comment-Date: Mon, 23 Jan 2023 18:24:46 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Wail Alkowaileet
Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Jenkins : Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Vijay Sarathy, Till Westmann. Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 ) Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. Patch Set 8: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/159/ : 1 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 Gerrit-Change-Number: 17313 Gerrit-PatchSet: 8 Gerrit-Owner: Vijay Sarathy Gerrit-Reviewer: Ali Alsuliman Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Vijay Sarathy Gerrit-Reviewer: Wail Alkowaileet Gerrit-Attention: Wail Alkowaileet Gerrit-Attention: Ali Alsuliman Gerrit-Attention: Vijay Sarathy Gerrit-Attention: Till Westmann Gerrit-Comment-Date: Sun, 22 Jan 2023 08:02:22 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Jenkins : Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Vijay Sarathy, Till Westmann. Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 ) Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. Patch Set 8: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/159/ -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 Gerrit-Change-Number: 17313 Gerrit-PatchSet: 8 Gerrit-Owner: Vijay Sarathy Gerrit-Reviewer: Ali Alsuliman Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Vijay Sarathy Gerrit-Reviewer: Wail Alkowaileet Gerrit-Attention: Wail Alkowaileet Gerrit-Attention: Ali Alsuliman Gerrit-Attention: Vijay Sarathy Gerrit-Attention: Till Westmann Gerrit-Comment-Date: Sun, 22 Jan 2023 07:11:54 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Vijay Sarathy : Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Vijay Sarathy, Till Westmann. Hello Wail Alkowaileet, Ali Alsuliman, Jenkins, Anon. E. Moose #1000171, Till Westmann, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 to look at the new patch set (#8). Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 --- M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.09.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/union/union_opt_1/union_opt_1.9.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate-sql/count_dataset/count_dataset.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/rebalance/single_dataset_with_index/single_dataset_with_index.8.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.17.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-lookup/push-limit-to-primary-lookup.3.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/pushdown-plans/pushdown-plans.04.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.11.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/explain/explain_object_constructor-02/explain_object_constructor-02.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan-select/push-limit-to-primary-scan-select.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/objects/load-record-fields/load-record-fields.4.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/rebalance/single_dataset_with_index/single_dataset_with_index.13.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.03.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-external-scan/push-limit-to-external-scan.2.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/misc/constant_folding/constant_folding.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate/count_dataset/count_dataset.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/pushdown-plans/pushdown-plans.03.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-external-scan-select/push-limit-to-external-scan-select.2.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan/push-limit-to-primary-scan.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan-select/push-limit-to-primary-scan-select.6.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/union/union_type_cast/union_type_cast.4.plan M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan/push-limit-to-primary-scan.7.plan M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitor.java M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-lookup-select/push-limit-to-primary-lookup-select.3.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/array-access-pushdown/array-access-pushdown.07.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/array-access-pushdown/array-access-pushdown.15.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/explain/explain_field_access_closed/explain_field_access_closed.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.15.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/list/enforcing_item_type/enforcing_item_type.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/array_fun/array_remove/array_remove.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-library/deterministic/deterministic.4.plan M
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Jenkins : Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Vijay Sarathy, Till Westmann. Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 ) Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. Patch Set 7: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/158/ -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 Gerrit-Change-Number: 17313 Gerrit-PatchSet: 7 Gerrit-Owner: Vijay Sarathy Gerrit-Reviewer: Ali Alsuliman Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Vijay Sarathy Gerrit-Reviewer: Wail Alkowaileet Gerrit-Attention: Wail Alkowaileet Gerrit-Attention: Ali Alsuliman Gerrit-Attention: Vijay Sarathy Gerrit-Attention: Till Westmann Gerrit-Comment-Date: Sun, 22 Jan 2023 01:35:51 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost/card to ...
>From Vijay Sarathy : Attention is currently required from: Wail Alkowaileet, Ali Alsuliman, Vijay Sarathy, Till Westmann. Hello Wail Alkowaileet, Ali Alsuliman, Jenkins, Anon. E. Moose #1000171, Till Westmann, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17313 to look at the new patch set (#7). Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan .. PLEASE EDIT to provide a meaningful commit message! Add cost/card to explain string plan Change-Id: Ie697b90cd4396208ab8f59c68e20ced236903fb8 --- M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.09.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/union/union_opt_1/union_opt_1.9.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate-sql/count_dataset/count_dataset.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/rebalance/single_dataset_with_index/single_dataset_with_index.8.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.17.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-lookup/push-limit-to-primary-lookup.3.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/pushdown-plans/pushdown-plans.04.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.11.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/explain/explain_object_constructor-02/explain_object_constructor-02.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan-select/push-limit-to-primary-scan-select.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/objects/load-record-fields/load-record-fields.4.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/rebalance/single_dataset_with_index/single_dataset_with_index.13.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.03.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-external-scan/push-limit-to-external-scan.2.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/misc/constant_folding/constant_folding.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/aggregate/count_dataset/count_dataset.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/pushdown-plans/pushdown-plans.03.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-external-scan-select/push-limit-to-external-scan-select.2.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan/push-limit-to-primary-scan.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan-select/push-limit-to-primary-scan-select.6.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/union/union_type_cast/union_type_cast.4.plan M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-scan/push-limit-to-primary-scan.7.plan M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitor.java M asterixdb/asterix-app/src/test/resources/runtimets/results/limit/push-limit-to-primary-lookup-select/push-limit-to-primary-lookup-select.3.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/array-access-pushdown/array-access-pushdown.07.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/array-access-pushdown/array-access-pushdown.15.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/explain/explain_field_access_closed/explain_field_access_closed.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/parquet/field-access-pushdown/field-access-pushdown.15.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/list/enforcing_item_type/enforcing_item_type.1.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/array_fun/array_remove/array_remove.5.plan M asterixdb/asterix-app/src/test/resources/runtimets/results/external-library/deterministic/deterministic.4.plan M
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message! Add cost and card...
>From Vijay Sarathy :
Vijay Sarathy has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17301 )
Change subject: PLEASE EDIT to provide a meaningful commit message! Add cost
and cardinality to explain
..
PLEASE EDIT to provide a meaningful commit message!
Add cost and cardinality to explain
Change-Id: I7b4a6c78dca3326f070e1dda888d774dc6100c0b
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/ExplainOnlyResultsPrinter.java
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EstimatedCostComputationVisitor.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ResultUtil.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
A
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/ExplainResults.java
6 files changed, 130 insertions(+), 27 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/01/17301/1
diff --git
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EstimatedCostComputationVisitor.java
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EstimatedCostComputationVisitor.java
index 9b9fd28..f9f34bf 100644
---
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EstimatedCostComputationVisitor.java
+++
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EstimatedCostComputationVisitor.java
@@ -86,7 +86,7 @@
public Pair
visitEmptyTupleSourceOperator(EmptyTupleSourceOperator op, Double arg)
throws AlgebricksException {
// Empty tuple source operator sends an empty tuple to downstream
operators.
-return new Pair<>(1.0, 1.0);
+return new Pair<>(0.0, 0.0);
}
@Override
@@ -220,7 +220,7 @@
@Override
public Pair visitDataScanOperator(DataSourceScanOperator
op, Double arg)
throws AlgebricksException {
-Pair cardCost = new Pair<>(1.0, 1.0);
+Pair cardCost = new Pair<>(0.0, 0.0);
for (Map.Entry anno : op.getAnnotations().entrySet()) {
if (anno.getValue() != null &&
anno.getKey().equals(OperatorAnnotations.OP_INPUT_CARDINALITY)) {
@@ -250,7 +250,7 @@
op.getAnnotations().put(OperatorAnnotations.OP_COST_LOCAL,
exchCost);
op.getAnnotations().put(OperatorAnnotations.OP_COST_TOTAL,
exchCost + cardCost.getSecond());
} else {
-op.getAnnotations().put(OperatorAnnotations.OP_COST_LOCAL,
cardCost.getSecond());
+op.getAnnotations().put(OperatorAnnotations.OP_COST_LOCAL, 0.0);
op.getAnnotations().put(OperatorAnnotations.OP_COST_TOTAL,
cardCost.getSecond());
}
op.getAnnotations().put(OperatorAnnotations.OP_OUTPUT_CARDINALITY,
cardCost.getFirst());
@@ -305,12 +305,12 @@
private static Pair
annotate(EstimatedCostComputationVisitor visitor, ILogicalOperator op,
Double arg) throws AlgebricksException {
if (op.getInputs().isEmpty()) {
-return new Pair<>(1.0, 1.0);
+return new Pair<>(0.0, 0.0);
}
Pair cardCost =
op.getInputs().get(0).getValue().accept(visitor, arg);
op.getAnnotations().put(OperatorAnnotations.OP_OUTPUT_CARDINALITY,
cardCost.getFirst());
op.getAnnotations().put(OperatorAnnotations.OP_COST_TOTAL,
cardCost.getSecond());
-op.getAnnotations().put(OperatorAnnotations.OP_COST_LOCAL,
cardCost.getSecond());
+op.getAnnotations().put(OperatorAnnotations.OP_COST_LOCAL, 0.0);
return cardCost;
}
@@ -322,9 +322,9 @@
return operator.getInputs().get(0).getValue().accept(this, arg);
}
-// Visits an inner join operator, particularly, deals with the case the
join is a cartesian product.
+// Visits an inner join operator.
private Pair visitInnerJoin(InnerJoinOperator
joinOperator, Double arg) throws AlgebricksException {
-Pair cardCost = new Pair<>(1.0, 1.0);
+Pair cardCost = new Pair<>(0.0, 0.0);
ILogicalOperator left = joinOperator.getInputs().get(0).getValue();
ILogicalOperator right = joinOperator.getInputs().get(1).getValue();
diff --git
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
index adcdcb2..3a06329 100644
---
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
+++
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
@@ -35,6 +35,7 @@
import
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Vijay Sarathy : Attention is currently required from: Vijay Sarathy. Hello Ali Alsuliman, Jenkins, Anon. E. Moose #1000171, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17299 to look at the new patch set (#5). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! [ASTERIXDB-3093][COMP] CBO enhancements and fixes after calibration tests. Change-Id: Ifa7da177ab94ba39ee2d3e24813eced48e3b8304 --- D asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/collocated.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q21.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/q08_group_by.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q8.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/filter_on_meta_with_idx_2.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/nested-open-index/btree-index-join/non-enforced-composite-key-equi-join/05.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/extract-common-operators/extract-common-operators.01.plan D asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/leftouterjoin/query-ASTERIXDB-2857.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-ternary-inlj/query4.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/open-index-non-enforced/btree-equi-join-non-enforced/btree-equi-join-non-enforced-07.plan M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/joins/nlj_partitioning_property_2.plan M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q9.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping_ps.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q5.plan M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q2.plan M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/q09_group_by.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpcds/query-ASTERIXDB-1580.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping_broadcast_ps.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpcds/query-ASTERIXDB-1591.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping_broadcast.plan D asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/denorm-cust-order.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/nested-index/btree-index-join/ASTERIXDB-2199.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/open-index-non-enforced/btree-equi-join-non-enforced/btree-equi-join-non-enforced-07_ps.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q7.plan M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/joins/nlj_partitioning_property_1.plan 31 files changed, 1,063 insertions(+), 907 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/99/17299/5 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17299 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ifa7da177ab94ba39ee2d3e24813eced48e3b8304 Gerrit-Change-Number: 17299 Gerrit-PatchSet: 5 Gerrit-Owner: Vijay Sarathy Gerrit-Reviewer: Ali Alsuliman Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Vijay Sarathy Gerrit-CC: Till Westmann Gerrit-CC: Till Westmann Gerrit-CC: murali4...@gmail.com Gerrit-Attention: Vijay Sarathy Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Sushrut Borkar : Sushrut Borkar has restored this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Restored -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Iba4f46a9d1ea7b8cadd7cca250d43576ad036438 Gerrit-Change-Number: 17294 Gerrit-PatchSet: 1 Gerrit-Owner: Sushrut Borkar Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-MessageType: restore
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Iba4f46a9d1ea7b8cadd7cca250d43576ad036438 Gerrit-Change-Number: 17294 Gerrit-PatchSet: 1 Gerrit-Owner: Sushrut Borkar Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Tue, 29 Nov 2022 05:12:13 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Sushrut Borkar : Sushrut Borkar has abandoned this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17294 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Iba4f46a9d1ea7b8cadd7cca250d43576ad036438 Gerrit-Change-Number: 17294 Gerrit-PatchSet: 1 Gerrit-Owner: Sushrut Borkar Gerrit-CC: Jenkins Gerrit-MessageType: abandon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Glenn Galvizo : Glenn Galvizo has abandoned this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17286 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17286 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I60054b49e61fc665dfb2aebf7087cb8d3cd1dcc0 Gerrit-Change-Number: 17286 Gerrit-PatchSet: 1 Gerrit-Owner: Glenn Galvizo Gerrit-MessageType: abandon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 4: Contrib-2 Analytics Compatibility Tests Failed : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 4 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Tue, 15 Nov 2022 00:38:04 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 5: Contrib-2 Analytics Compatibility Tests Failed : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 5 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Tue, 15 Nov 2022 00:32:27 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Attention is currently required from: murali4...@gmail.com. Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 5: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/50/ -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 5 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Mon, 14 Nov 2022 22:59:54 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 4: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 4 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Mon, 14 Nov 2022 22:20:04 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 5: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 5 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Mon, 14 Nov 2022 22:19:14 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From : Attention is currently required from: murali4...@gmail.com. Hello Jenkins, Anon. E. Moose #1000171, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 to look at the new patch set (#5). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit 8c3d0e6f8ad65c1c7b25b610885a5268b51abd9d Author: murali4104 Date: Tue Nov 8 10:29:59 2022 -0800 back to JnI and JnJ commit 755b2912e3a5cf95636a7322b6f06b082e8622bf Author: murali4104 Date: Tue Nov 8 10:06:29 2022 -0800 multiple nested loop changes by pusing assigns out of internal edges and not modifying the join predicate commit 4f6bf520402c0b13d098c899ae925c7d30ec5672 Author: murali4104 Date: Tue Nov 1 13:38:07 2022 -0700 added LOWEST COST printing commit 46954eb73b240e673b5a86561fcfc92119321b10 Author: murali4104 Date: Thu Oct 27 16:29:27 2022 -0700 selec. hint will not be overwritten commit 44789c513655b6f0ab389c2681b18c9c2406d87f Author: Ali Alsuliman Date: Tue Sep 27 13:08:15 2022 -0700 WIP: join hints Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/CostMethods.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/Stats.java M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/util/JoinUtils.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java 10 files changed, 446 insertions(+), 95 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/17278/5 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 5 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From : Attention is currently required from: murali4...@gmail.com. Hello Jenkins, Anon. E. Moose #1000171, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 to look at the new patch set (#4). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit 8c3d0e6f8ad65c1c7b25b610885a5268b51abd9d Author: murali4104 Date: Tue Nov 8 10:29:59 2022 -0800 back to JnI and JnJ commit 755b2912e3a5cf95636a7322b6f06b082e8622bf Author: murali4104 Date: Tue Nov 8 10:06:29 2022 -0800 multiple nested loop changes by pusing assigns out of internal edges and not modifying the join predicate commit 4f6bf520402c0b13d098c899ae925c7d30ec5672 Author: murali4104 Date: Tue Nov 1 13:38:07 2022 -0700 added LOWEST COST printing commit 46954eb73b240e673b5a86561fcfc92119321b10 Author: murali4104 Date: Thu Oct 27 16:29:27 2022 -0700 selec. hint will not be overwritten commit 44789c513655b6f0ab389c2681b18c9c2406d87f Author: Ali Alsuliman Date: Tue Sep 27 13:08:15 2022 -0700 WIP: join hints Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/CostMethods.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/Stats.java M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/util/JoinUtils.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java 10 files changed, 446 insertions(+), 95 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/17278/4 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 4 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Contrib-2 Analytics Compatibility Tests Failed : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 3 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Fri, 11 Nov 2022 01:42:00 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 2: Contrib-2 Analytics Compatibility Tests Failed : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 2 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Fri, 11 Nov 2022 01:25:27 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Attention is currently required from: murali4...@gmail.com. Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/45/ -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 3 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Thu, 10 Nov 2022 23:42:29 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 2: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 2 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Thu, 10 Nov 2022 23:09:32 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 3 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Thu, 10 Nov 2022 23:08:06 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From : Attention is currently required from: murali4...@gmail.com. Hello Jenkins, Anon. E. Moose #1000171, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 to look at the new patch set (#3). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit 8c3d0e6f8ad65c1c7b25b610885a5268b51abd9d Author: murali4104 Date: Tue Nov 8 10:29:59 2022 -0800 back to JnI and JnJ commit 755b2912e3a5cf95636a7322b6f06b082e8622bf Author: murali4104 Date: Tue Nov 8 10:06:29 2022 -0800 multiple nested loop changes by pusing assigns out of internal edges and not modifying the join predicate commit 4f6bf520402c0b13d098c899ae925c7d30ec5672 Author: murali4104 Date: Tue Nov 1 13:38:07 2022 -0700 added LOWEST COST printing commit 46954eb73b240e673b5a86561fcfc92119321b10 Author: murali4104 Date: Thu Oct 27 16:29:27 2022 -0700 selec. hint will not be overwritten commit 44789c513655b6f0ab389c2681b18c9c2406d87f Author: Ali Alsuliman Date: Tue Sep 27 13:08:15 2022 -0700 WIP: join hints Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/CostMethods.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/Stats.java M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/util/JoinUtils.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java 10 files changed, 446 insertions(+), 95 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/17278/3 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 3 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From : Attention is currently required from: murali4...@gmail.com. Hello Jenkins, Anon. E. Moose #1000171, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 to look at the new patch set (#2). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit 8c3d0e6f8ad65c1c7b25b610885a5268b51abd9d Author: murali4104 Date: Tue Nov 8 10:29:59 2022 -0800 back to JnI and JnJ commit 755b2912e3a5cf95636a7322b6f06b082e8622bf Author: murali4104 Date: Tue Nov 8 10:06:29 2022 -0800 multiple nested loop changes by pusing assigns out of internal edges and not modifying the join predicate commit 4f6bf520402c0b13d098c899ae925c7d30ec5672 Author: murali4104 Date: Tue Nov 1 13:38:07 2022 -0700 added LOWEST COST printing commit 46954eb73b240e673b5a86561fcfc92119321b10 Author: murali4104 Date: Thu Oct 27 16:29:27 2022 -0700 selec. hint will not be overwritten commit 44789c513655b6f0ab389c2681b18c9c2406d87f Author: Ali Alsuliman Date: Tue Sep 27 13:08:15 2022 -0700 WIP: join hints Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/PlanNode.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/CostMethods.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/Stats.java M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/util/JoinUtils.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/EnumerateJoinsRule.java 10 files changed, 446 insertions(+), 95 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/17278/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 2 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Attention is currently required from: murali4...@gmail.com. Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Contrib-2 Analytics Compatibility Tests Failed : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 1 Gerrit-Owner: murali4...@gmail.com Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Attention: murali4...@gmail.com Gerrit-Comment-Date: Tue, 08 Nov 2022 21:16:26 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests-redux/43/ -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 1 Gerrit-Owner: murali4...@gmail.com Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Tue, 08 Nov 2022 19:17:02 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17278 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Id9b147de07311a3cfd0c6b57887eb218bbcf4dac Gerrit-Change-Number: 17278 Gerrit-PatchSet: 1 Gerrit-Owner: murali4...@gmail.com Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Tue, 08 Nov 2022 18:44:01 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17184 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/sC7wr6jfSrfGpTrF9 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17184 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ib803da4f72c96290bba972a18b35717d9fe86b5c Gerrit-Change-Number: 17184 Gerrit-PatchSet: 1 Gerrit-Owner: Vijay Sarathy Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Tue, 23 Aug 2022 17:02:22 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Vijay Sarathy : Vijay Sarathy has abandoned this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17184 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17184 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ib803da4f72c96290bba972a18b35717d9fe86b5c Gerrit-Change-Number: 17184 Gerrit-PatchSet: 1 Gerrit-Owner: Vijay Sarathy Gerrit-CC: Jenkins Gerrit-MessageType: abandon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Vijay Sarathy : Vijay Sarathy has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17184 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit f359a439724681cdd1ee3aae2c0c4d51c2dda3b5 Author: Vijay Sarathy Date: Mon Aug 22 12:52:25 2022 -0700 [ASTERIXDB-3046][COMP] Support cost based query optimization. - user model changes: yes - storage format changes: no - interface changes: yes Details: Cost based query optimization enables the asterixdb optimizer to compute the optimal plan for a query. commit 296600b5873462796ea6790461945a4cf369135b Author: Vijay Sarathy Date: Thu Jun 16 16:30:12 2022 -0700 [ASTERIXDB-3046][COMP] Changes to support cost based query optimization in Asterix DB. - user model changes: no - storage format changes: no - interface changes: no Details: Cost based query optimization enables the asterixdb optimizer to compute the optimal plan for a query. Change-Id: Ib803da4f72c96290bba972a18b35717d9fe86b5c --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/DefaultRuleSetFactory.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/Cost.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/CostMethods.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/ICost.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/cost/ICostMethods.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/EnumerateJoinsRule.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IAccessMethod.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IntroduceJoinAccessMethodRule.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IntroduceSelectAccessMethodRule.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/InvertedIndexAccessMethod.java M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/RTreeAccessMethod.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/util/JoinCondition.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/util/JoinEnum.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/util/JoinNode.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/util/PlanNode.java A asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/util/StatsUtil.java M asterixdb/asterix-app/pom.xml M asterixdb/asterix-app/src/main/resources/cc.conf M asterixdb/asterix-app/src/main/resources/cc2.conf M asterixdb/asterix-app/src/main/resources/cc3.conf A asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/optimizer/OptimizerTest.java M asterixdb/asterix-app/src/test/resources/cc-compression.conf M asterixdb/asterix-app/src/test/resources/cc-ssl.conf M asterixdb/asterix-app/src/test/resources/optimizerts/results/spatial_join/st_distance_static_partitioning.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-join/hints-indexnl-params/hints-indexnl-params-3.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-join/hints-indexnl-params/hints-indexnl-params-5.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-join/hints-indexnl-params/hints-indexnl-params-6.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-join/hints-indexnl-params/hints-indexnl-params-7.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-selection/disjunctive-predicate/disjunctive-predicate-3.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-index-selection/hints-skip-index/hints-skip-index-11.plan A asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/btree-ternary-inlj/query4.plan A
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Contrib-2 Analytics Compatibility Tests Failed https://cbjenkins.page.link/vfU15aasNGtRKPuM7 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 Gerrit-Change-Number: 12967 Gerrit-PatchSet: 3 Gerrit-Owner: Shiva Jahangiri Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin Gerrit-Reviewer: Glenn Galvizo Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Comment-Date: Thu, 02 Sep 2021 16:10:36 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/12441/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 Gerrit-Change-Number: 12967 Gerrit-PatchSet: 3 Gerrit-Owner: Shiva Jahangiri Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin Gerrit-Reviewer: Glenn Galvizo Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Comment-Date: Thu, 02 Sep 2021 15:43:12 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 3: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/zzfFUywNafhxyixm8 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 Gerrit-Change-Number: 12967 Gerrit-PatchSet: 3 Gerrit-Owner: Shiva Jahangiri Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin Gerrit-Reviewer: Glenn Galvizo Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Comment-Date: Thu, 02 Sep 2021 14:00:02 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Shiva Jahangiri : Hello Glenn Galvizo, Anon. E. Moose #1000171, Till Westmann, Jenkins, Ian Maxon, Dmitry Lychagin, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 to look at the new patch set (#3). Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! [NO ISSUE] Added Wisconsin Data Generator for tests - user model changes: no - storage format changes: no - interface changes: no Details: - Wisconsin Data Generator has been added as a separate package to generate data for related test queries. Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 --- M asterixdb/asterix-app/pom.xml A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/LICENSE A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/README.md A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/pom.xml A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Constants/DataTypes.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Constants/Order.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Constants/StringWordList.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/FieldGenerators/AWisconsinGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/FieldGenerators/WisconsinBinaryGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/FieldGenerators/WisconsinNumberGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/FieldGenerators/WisconsinStringGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/FieldGenerators/ZipfGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/OutputGenerator/AWisconsinOutputGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/OutputGenerator/WisconsinAsterixDBLoadOutputGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/OutputGenerator/WisconsinFileOutputGenerator.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Parser/PField.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Parser/Parser.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Schema/Field.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Schema/Schema.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Server.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/java/com/datagen/Utils/Utils.java A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/resources/log4j2.xml A asterixdb/asterix-app/src/test/java/org/apache/asterix/wisconsindatagenerator/src/main/resources/wisconsin_datagen.properties 23 files changed, 2,367 insertions(+), 0 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/67/12967/3 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 Gerrit-Change-Number: 12967 Gerrit-PatchSet: 3 Gerrit-Owner: Shiva Jahangiri Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin Gerrit-Reviewer: Glenn Galvizo Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-MessageType: newpatchset
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/w4p9txcKt7j1EGLb6 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12967 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie13be468ce2de60200864759a7eca4550132a2a3 Gerrit-Change-Number: 12967 Gerrit-PatchSet: 1 Gerrit-Owner: Shiva Jahangiri Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Wed, 25 Aug 2021 05:25:58 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12404 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration-Tests-1 Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/12240/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12404 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I7997156936612bd068285710eacb5f6f8d0e0d3c Gerrit-Change-Number: 12404 Gerrit-PatchSet: 1 Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Tue, 20 Jul 2021 01:48:36 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12404 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Contrib-2 Analytics Compatibility Compilation Failed https://cbjenkins.page.link/c23YKBVJTLRMNNwE6 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12404 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I7997156936612bd068285710eacb5f6f8d0e0d3c Gerrit-Change-Number: 12404 Gerrit-PatchSet: 1 Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Tue, 20 Jul 2021 00:33:31 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Glenn Galvizo : Glenn Galvizo has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12003 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit 1883713fd31628b3aa6854c1adfadb8a5bd69343 Author: ggalvizo Date: Sun Jun 13 01:12:26 2021 -0700 In order to get this to compile on my M1 macbook, this plugin must be updated. Change-Id: Ie947924ddff4da62c5e95f6a93ad827550a6fe67 --- M asterixdb/asterix-dashboard/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/03/12003/1 diff --git a/asterixdb/asterix-dashboard/pom.xml b/asterixdb/asterix-dashboard/pom.xml index 8bf3389..7937f75 100644 --- a/asterixdb/asterix-dashboard/pom.xml +++ b/asterixdb/asterix-dashboard/pom.xml @@ -66,7 +66,7 @@ com.github.eirslett frontend-maven-plugin -1.6 +1.11.0 v14.15.4 6.14.11 -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12003 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: Ie947924ddff4da62c5e95f6a93ad827550a6fe67 Gerrit-Change-Number: 12003 Gerrit-PatchSet: 1 Gerrit-Owner: Glenn Galvizo Gerrit-MessageType: newchange
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Jenkins : Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/11723 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/12033/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/11723 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I328bf4e80037a23b6d44139548436f6c4f232bba Gerrit-Change-Number: 11723 Gerrit-PatchSet: 1 Gerrit-Owner: Glenn Galvizo Gerrit-Reviewer: Jenkins Gerrit-CC: Anon. E. Moose #1000171 Gerrit-Comment-Date: Wed, 02 Jun 2021 21:21:09 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/11723 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/zYbieJQarZH3SyXt5 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/11723 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I328bf4e80037a23b6d44139548436f6c4f232bba Gerrit-Change-Number: 11723 Gerrit-PatchSet: 1 Gerrit-Owner: Glenn Galvizo Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Wed, 02 Jun 2021 20:27:36 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Glenn Galvizo :
Glenn Galvizo has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/11723 )
Change subject: PLEASE EDIT to provide a meaningful commit message!
..
PLEASE EDIT to provide a meaningful commit message!
The following commits from your working branch will be included:
commit 8cb9a782df3ec1c8e42a59341d7d7b37d0eb5906
Author: ggalvizo
Date: Sat May 29 19:43:34 2021 -0700
Currently using the try-catch pattern to implement the implicit DISTINCT
for array index maintenance.
Change-Id: I328bf4e80037a23b6d44139548436f6c4f232bba
---
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryUpsertWithNestedPlanOperatorNodePushable.java
3 files changed, 66 insertions(+), 49 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/23/11723/1
diff --git
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
index c3859be..182687d 100644
---
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
+++
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
@@ -449,7 +449,7 @@
context.computeAndSetTypeEnvironmentForOperator(unnestSourceOp);
UnnestBranchCreator unnestSIDXBranch =
buildUnnestBranch(unnestSourceOp, index, newRecordVar,
newMetaVar, recType, metaType,
dataset.hasMetaPart());
-unnestSIDXBranch.applyProjectDistinct();
+unnestSIDXBranch.applyProjectOnly();
// If there exists a filter expression, add it to the
top of our nested plan.
filterExpression =
(primaryIndexModificationOp.getOperation() == Kind.UPSERT) ? null
@@ -477,7 +477,7 @@
UnnestBranchCreator unnestBeforeSIDXBranch =
buildUnnestBranch(unnestBeforeSourceOp, index,
primaryIndexModificationOp.getBeforeOpRecordVar(), beforeOpMetaVar, recType,
metaType, dataset.hasMetaPart());
-unnestBeforeSIDXBranch.applyProjectDistinct();
+unnestBeforeSIDXBranch.applyProjectOnly();
indexUpdate.getNestedPlans().add(unnestBeforeSIDXBranch.buildBranch());
}
} else if (index.getIndexType() == IndexType.ARRAY &&
isBulkload) {
@@ -1010,6 +1010,13 @@
return varRef;
}
+public final void applyProjectOnly() throws AlgebricksException {
+List projectVars = new
ArrayList<>(this.lastFieldVars);
+ProjectOperator projectOperator = new ProjectOperator(projectVars);
+projectOperator.setSourceLocation(sourceLoc);
+this.currentTop = introduceNewOp(currentTop, projectOperator,
true);
+}
+
@SafeVarargs
public final void
applyProjectDistinct(List>... auxiliaryExpressions)
throws AlgebricksException {
diff --git
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
index 0b36774..99b9a08 100644
---
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
+++
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
@@ -31,13 +31,13 @@
import org.apache.hyracks.api.context.IHyracksTaskContext;
import org.apache.hyracks.api.dataflow.value.RecordDescriptor;
import org.apache.hyracks.api.exceptions.HyracksDataException;
-import org.apache.hyracks.dataflow.common.comm.io.ArrayTupleBuilder;
-import org.apache.hyracks.dataflow.common.comm.io.ArrayTupleReference;
import org.apache.hyracks.dataflow.common.comm.io.FrameTupleAccessor;
import org.apache.hyracks.dataflow.common.comm.util.FrameUtils;
+import org.apache.hyracks.dataflow.common.data.accessors.FrameTupleReference;
import
org.apache.hyracks.storage.am.common.api.IModificationOperationCallbackFactory;
import
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10229 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/6v7pzvyL422cGRgDA : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10229 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I74341fa8cfb63c85590877cb432d4c3e7c857268 Gerrit-Change-Number: 10229 Gerrit-PatchSet: 1 Gerrit-Owner: Tin Vu Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Fri, 26 Feb 2021 21:14:26 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
>From Tin Vu : Tin Vu has abandoned this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10229 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10229 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I74341fa8cfb63c85590877cb432d4c3e7c857268 Gerrit-Change-Number: 10229 Gerrit-PatchSet: 1 Gerrit-Owner: Tin Vu Gerrit-CC: Jenkins Gerrit-MessageType: abandon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10227 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/tYihq6YEWpAQ66kYA : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10227 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I9750336eb24205909090a22c33391fca2c2ab039 Gerrit-Change-Number: 10227 Gerrit-PatchSet: 1 Gerrit-Owner: Tin Vu Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Fri, 26 Feb 2021 20:41:06 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10226 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/CXR7Bvz2BvutHSrMA : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/10226 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I2ac7449a86635ae70fc642f4bd596808f3ad389a Gerrit-Change-Number: 10226 Gerrit-PatchSet: 1 Gerrit-Owner: Tin Vu Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins Gerrit-Comment-Date: Fri, 26 Feb 2021 20:35:21 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/4362/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Anon. E. Moose (1000171) Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:49:26 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5941/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:10 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3731/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:10 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose (1000171) has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://cbjenkins.page.link/CWHFyhhwji2299Tu5 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Anon. E. Moose (1000171) Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:35:15 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/4362/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:32:36 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8498/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:31:38 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5718/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:25 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5734/ (14/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:30 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5697/ (16/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:35 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/11208/ (15/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:32 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/660/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:27 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/499/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:28 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/995/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:14 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/1087/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:17 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/897/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:19 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/6301/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:13 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/6091/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:21 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9677/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:12 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3383 ) Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5634/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3383 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id0257d3895ce902461ebcc61d95ec44e65878275 Gerrit-Change-Number: 3383 Gerrit-PatchSet: 1 Gerrit-Owner: Anon. E. Moose Gerrit-Reviewer: Jenkins Gerrit-Comment-Date: Mon, 06 May 2019 16:29:10 + Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10978/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5713/ (14/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5403/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/6073/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/861/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5492/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Ian Maxon has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/3314
Change subject: PLEASE EDIT to provide a meaningful commit message!
..
PLEASE EDIT to provide a meaningful commit message!
The following commits from your working branch will be included:
commit 5bffb3da4ce8b876631407836386009098582f0c
Merge: 111d347252 45d9395585
Author: Ian Maxon
Date: Wed Apr 3 10:27:22 2019 -0700
Merge branch 'master' into failify
commit 111d3472526c73520b738be77cca2de71dcc2483
Author: Armin Balalaie
Date: Wed Apr 3 00:27:32 2019 -0700
[NO ISSUE] Changes SpiderSilk to Failify
- user model changes: no
- storage format changes: no
- interface changes: no
Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07
---
M .gitignore
R asterixdb/asterix-failify/config/cc.conf
R asterixdb/asterix-failify/docker/Dockerfile
R asterixdb/asterix-failify/pom.xml
R
asterixdb/asterix-failify/src/test/java/org/apache/asterix/failify/SampleTestIT.java
A
asterixdb/asterix-failify/src/test/java/org/apache/asterix/failify/TestUtil.java
R asterixdb/asterix-failify/src/test/resources/logback.xml
D
asterixdb/asterix-spidersilk/src/test/java/org/apache/asterix/spidersilk/TestUtil.java
M asterixdb/pom.xml
9 files changed, 102 insertions(+), 94 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/14/3314/1
diff --git a/.gitignore b/.gitignore
index 1dcc3c2..62d449f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -28,4 +28,4 @@
.m2*
ß
-.SpiderSilkWorkingDirectory
+.FailifyWorkingDirectory
diff --git a/asterixdb/asterix-spidersilk/config/cc.conf
b/asterixdb/asterix-failify/config/cc.conf
similarity index 100%
rename from asterixdb/asterix-spidersilk/config/cc.conf
rename to asterixdb/asterix-failify/config/cc.conf
diff --git a/asterixdb/asterix-spidersilk/docker/Dockerfile
b/asterixdb/asterix-failify/docker/Dockerfile
similarity index 100%
rename from asterixdb/asterix-spidersilk/docker/Dockerfile
rename to asterixdb/asterix-failify/docker/Dockerfile
diff --git a/asterixdb/asterix-spidersilk/pom.xml
b/asterixdb/asterix-failify/pom.xml
similarity index 95%
rename from asterixdb/asterix-spidersilk/pom.xml
rename to asterixdb/asterix-failify/pom.xml
index 6b370c8..9cb2e7f 100644
--- a/asterixdb/asterix-spidersilk/pom.xml
+++ b/asterixdb/asterix-failify/pom.xml
@@ -18,8 +18,8 @@
!-->
http://maven.apache.org/POM/4.0.0;
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance;
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
http://maven.apache.org/maven-v4_0_0.xsd;>
4.0.0
- asterix-spidersilk
- asterix-spidersilk
+ asterix-failify
+ asterix-failify
org.apache.asterix
@@ -57,9 +57,9 @@
test
- me.arminb.spidersilk
- spidersilk
- 0.4.0
+ io.failify
+ failify
+ 0.2.0
test
diff --git
a/asterixdb/asterix-spidersilk/src/test/java/org/apache/asterix/spidersilk/SampleTestIT.java
b/asterixdb/asterix-failify/src/test/java/org/apache/asterix/failify/SampleTestIT.java
similarity index 91%
rename from
asterixdb/asterix-spidersilk/src/test/java/org/apache/asterix/spidersilk/SampleTestIT.java
rename to
asterixdb/asterix-failify/src/test/java/org/apache/asterix/failify/SampleTestIT.java
index b3b73bf..8f1bd59 100644
---
a/asterixdb/asterix-spidersilk/src/test/java/org/apache/asterix/spidersilk/SampleTestIT.java
+++
b/asterixdb/asterix-failify/src/test/java/org/apache/asterix/failify/SampleTestIT.java
@@ -16,14 +16,15 @@
* specific language governing permissions and limitations
* under the License.
*/
-package org.apache.asterix.spidersilk;
+package org.apache.asterix.failify;
-import java.io.InputStream;
-import java.util.List;
-import java.util.Map;
-
+import com.fasterxml.jackson.databind.ObjectMapper;
+import io.failify.FailifyRunner;
+import io.failify.dsl.entities.Deployment;
+import io.failify.exceptions.RuntimeEngineException;
import org.apache.asterix.test.common.TestExecutor;
import org.apache.asterix.testframework.context.TestCaseContext;
+
import org.junit.AfterClass;
import org.junit.Assert;
import org.junit.BeforeClass;
@@ -31,21 +32,20 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.fasterxml.jackson.databind.ObjectMapper;
+import java.io.InputStream;
-import me.arminb.spidersilk.SpiderSilkRunner;
-import me.arminb.spidersilk.dsl.entities.Deployment;
-import me.arminb.spidersilk.exceptions.RuntimeEngineException;
+import java.util.List;
+import java.util.Map;
public class SampleTestIT {
private static final Logger logger =
LoggerFactory.getLogger(SampleTestIT.class);
-protected static SpiderSilkRunner runner;
+protected static FailifyRunner runner;
@BeforeClass
public static void before() throws RuntimeEngineException {
Deployment deployment = TestUtil.getSimpleClusterDeployment();
-runner =
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/5671/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9447/ (15/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5460/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/669/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/435/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Ian Maxon has abandoned this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5861/ (16/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/281/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5498/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3504/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/776/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3314 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e9efa2c14fafbe5981fbb2fed77533173d5cc07 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5481/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3312 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic2371171dfc01423921c2e2d04fbc68a38978f2e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/659/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3312 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic2371171dfc01423921c2e2d04fbc68a38978f2e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/3312
Change subject: PLEASE EDIT to provide a meaningful commit message!
..
PLEASE EDIT to provide a meaningful commit message!
The following commits from your working branch will be included:
commit 4ce0cbbfde42525bf005bfbef7aa5ba5d0a9707c
Author: Michael Blow
Date: Tue Apr 2 21:31:43 2019 -0400
foo
commit fe8f855711a9fe97056b62149151b7739de21a46
Author: Michael Blow
Date: Tue Apr 2 21:27:06 2019 -0400
foo
commit 235b75839da74b32df56075a45c72ae56138b79c
Author: Michael Blow
Date: Tue Apr 2 21:19:15 2019 -0400
foo
commit 28dc2f3ed10a3fbc4b8cb8d2596b621dea465779
Author: Michael Blow
Date: Tue Apr 2 20:46:50 2019 -0400
foo
commit 9b1706a4247f1279f3e19d8c405949cdbb108b0d
Author: Michael Blow
Date: Tue Apr 2 20:39:02 2019 -0400
foo
commit 6324221b21116f8cad61b67559cfd42712a71921
Author: Michael Blow
Date: Tue Apr 2 20:37:47 2019 -0400
foo
commit 0ece97ef487ffe21c560296d59e570a2831577aa
Author: Michael Blow
Date: Tue Apr 2 20:19:25 2019 -0400
foo
Change-Id: Ic2371171dfc01423921c2e2d04fbc68a38978f2e
---
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/NodeControllerData.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/NodeRegistration.java
R
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/resources/log4j2.xml
M
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/PidHelper.java
M
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/trace/Event.java
M
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/trace/Tracer.java
6 files changed, 20 insertions(+), 10 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/12/3312/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/NodeControllerData.java
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/NodeControllerData.java
index c37acab..24a3e57 100644
---
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/NodeControllerData.java
+++
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/NodeControllerData.java
@@ -79,7 +79,7 @@
private final Map systemProperties;
-private final int pid;
+private final long pid;
private final HeartbeatSchema hbSchema;
diff --git
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/NodeRegistration.java
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/NodeRegistration.java
index 437b001..474bc0a 100644
---
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/NodeRegistration.java
+++
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/NodeRegistration.java
@@ -75,7 +75,7 @@
private final NetworkAddress messagingPort;
-private final int pid;
+private final long pid;
private final NodeCapacity capacity;
@@ -190,7 +190,7 @@
return messagingPort;
}
-public int getPid() {
+public long getPid() {
return pid;
}
}
diff --git a/asterixdb/asterix-app/src/main/resources/log4j2.xml
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/resources/log4j2.xml
similarity index 100%
rename from asterixdb/asterix-app/src/main/resources/log4j2.xml
rename to
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/resources/log4j2.xml
diff --git
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/PidHelper.java
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/PidHelper.java
index 5a8edbd..46e77e3 100644
---
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/PidHelper.java
+++
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/PidHelper.java
@@ -21,6 +21,7 @@
import java.lang.management.ManagementFactory;
import java.lang.management.RuntimeMXBean;
import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import org.apache.logging.log4j.Level;
@@ -34,11 +35,20 @@
private PidHelper() {
}
-public static int getPid() {
+public static long getPid() {
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5393/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3312 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic2371171dfc01423921c2e2d04fbc68a38978f2e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5487/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3312 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic2371171dfc01423921c2e2d04fbc68a38978f2e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7840/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/7840/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Anon. E. Moose #1000171 has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Analytics Compatibility Compilation Successful https://goo.gl/Cv4N3e : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/417/ (8/15) -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5586/ (4/15) -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/4926/ (1/15) -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/4980/ (6/15) -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!
Jenkins has posted comments on this change. Change subject: PLEASE EDIT to provide a meaningful commit message! .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/418/ (2/15) -- To view, visit https://asterix-gerrit.ics.uci.edu/3137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b7b526457994b05a6823b01f7293a1a343e6a13 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
