Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has abandoned this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2596/ .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/2597 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I08c21b46ac6c5d1af93226532a5359532550567b Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2597 Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2596/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2596/ Change-Id: I08c21b46ac6c5d1af93226532a5359532550567b --- M pom.xml 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/97/2597/1 diff --git a/pom.xml b/pom.xml index c953683..8c19135 100644 --- a/pom.xml +++ b/pom.xml @@ -24,8 +24,9 @@ pom org.apache.asterix -apache-asterixdb +asterix-parent 0.9.4-SNAPSHOT +../asterix-parent/pom.xml asterix-bad -- To view, visit https://asterix-gerrit.ics.uci.edu/2597 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I08c21b46ac6c5d1af93226532a5359532550567b Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael Blow
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2596/ .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/267/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2597 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08c21b46ac6c5d1af93226532a5359532550567b Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has submitted this change and it was merged. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ Change-Id: I0fbc7ee17540b137353a14d0fbac575c36df30cd --- M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ProcedureDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/runtime/NotifyBrokerRuntime.java 10 files changed, 10 insertions(+), 10 deletions(-) Approvals: Jenkins: Verified Michael Blow: Looks good to me, approved diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java index 485a1e1..4f68f40 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java @@ -86,7 +86,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java index 3d43ee6..8403fcc 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java @@ -150,7 +150,7 @@ if (txnActive) { QueryTranslator.abort(e, e, mdTxnCtx); } -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java index 5f08845..ca1241c 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java @@ -219,7 +219,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java index 79c8efd..1b18f83 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java @@ -143,7 +143,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java index 99aded8..367599b 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java @@ -93,7 +93,7 @@
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2537 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0fbc7ee17540b137353a14d0fbac575c36df30cd Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/258/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2537 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0fbc7ee17540b137353a14d0fbac575c36df30cd Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2537 Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2533/ Change-Id: I0fbc7ee17540b137353a14d0fbac575c36df30cd --- M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateProcedureStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ProcedureDropStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/runtime/NotifyBrokerRuntime.java 10 files changed, 10 insertions(+), 10 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/37/2537/1 diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java index 485a1e1..4f68f40 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/BrokerDropStatement.java @@ -86,7 +86,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java index 3d43ee6..8403fcc 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelDropStatement.java @@ -150,7 +150,7 @@ if (txnActive) { QueryTranslator.abort(e, e, mdTxnCtx); } -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java index 5f08845..ca1241c 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelSubscribeStatement.java @@ -219,7 +219,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java index 79c8efd..1b18f83 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ChannelUnsubscribeStatement.java @@ -143,7 +143,7 @@ MetadataManager.INSTANCE.commitTransaction(mdTxnCtx); } catch (Exception e) { QueryTranslator.abort(e, e, mdTxnCtx); -throw new HyracksDataException(e); +throw HyracksDataException.create(e); } finally { metadataProvider.getLocks().unlock(); } diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java index 99aded8..367599b 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateBrokerStatement.java @@ -93,7
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has submitted this change and it was merged. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ - makes fuzzyjoin functions available to BAD tests Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e --- M asterix-bad/pom.xml 1 file changed, 7 insertions(+), 9 deletions(-) Approvals: abdullah alamoudi: Looks good to me, approved Jenkins: Verified diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml index 992f844..6f5cd32 100644 --- a/asterix-bad/pom.xml +++ b/asterix-bad/pom.xml @@ -154,6 +154,7 @@ org.apache.hadoop:hadoop-minicluster + org.apache.asterix:asterix-fuzzyjoin org.apache.asterix:asterix-common @@ -233,12 +234,10 @@ org.apache.hyracks algebricks-common - ${hyracks.version} org.apache.hyracks hyracks-dataflow-common - ${hyracks.version} org.apache.asterix @@ -253,32 +252,26 @@ log4j log4j - 1.2.17 org.apache.commons commons-lang3 - 3.5 org.apache.hyracks algebricks-core - ${hyracks.version} junit junit - 4.12 org.apache.hyracks hyracks-data-std - ${hyracks.version} org.apache.hyracks algebricks-runtime - ${hyracks.version} org.apache.asterix @@ -291,9 +284,14 @@ ${asterix.version} + org.apache.asterix + asterix-fuzzyjoin + ${asterix.version} + + + org.apache.hyracks hyracks-api - ${hyracks.version} -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Xikui Wang Gerrit-Reviewer: abdullah alamoudi
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
abdullah alamoudi has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Xikui Wang Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2481 to look at the new patch set (#2). Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ - makes fuzzyjoin functions available to BAD tests Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e --- M asterix-bad/pom.xml 1 file changed, 7 insertions(+), 9 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/81/2481/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Xikui Wang Gerrit-Reviewer: abdullah alamoudi
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/256/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Xikui Wang Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/255/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2481 Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/ - makes fuzzyjoin functions available to BAD tests Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e --- M asterix-bad/pom.xml 1 file changed, 7 insertions(+), 9 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/81/2481/1 diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml index 992f844..2a8b7e3 100644 --- a/asterix-bad/pom.xml +++ b/asterix-bad/pom.xml @@ -154,6 +154,7 @@ org.apache.hadoop:hadoop-minicluster + org.apache.asterix:asterix-fuzzyjoin org.apache.asterix:asterix-common @@ -233,12 +234,10 @@ org.apache.hyracks algebricks-common - ${hyracks.version} org.apache.hyracks hyracks-dataflow-common - ${hyracks.version} org.apache.asterix @@ -253,32 +252,26 @@ log4j log4j - 1.2.17 org.apache.commons commons-lang3 - 3.5 org.apache.hyracks algebricks-core - ${hyracks.version} junit junit - 4.12 org.apache.hyracks hyracks-data-std - ${hyracks.version} org.apache.hyracks algebricks-runtime - ${hyracks.version} org.apache.asterix @@ -291,9 +284,14 @@ ${asterix.version} + org.apache.asterix + asterix-fuzzyjoin + ${asterix.version} + test + + org.apache.hyracks hyracks-api - ${hyracks.version} -- To view, visit https://asterix-gerrit.ics.uci.edu/2481 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael Blow
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has submitted this change and it was merged. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 --- M asterix-bad/pom.xml M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java 3 files changed, 14 insertions(+), 24 deletions(-) Approvals: Jenkins: Verified Murtadha Hubail: Looks good to me, approved diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml index 291697d..20e97b2 100644 --- a/asterix-bad/pom.xml +++ b/asterix-bad/pom.xml @@ -282,22 +282,12 @@ org.apache.asterix - asterix-lang-common - ${asterix.version} - - - org.apache.asterix asterix-metadata ${asterix.version} org.apache.asterix asterix-external-data - ${asterix.version} - - - org.apache.asterix - asterix-transactions ${asterix.version} diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java index feaa3ca..53baf6d 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java @@ -40,6 +40,7 @@ import org.apache.asterix.bad.metadata.Channel; import org.apache.asterix.bad.metadata.DeployedJobSpecEventListener; import org.apache.asterix.bad.metadata.DeployedJobSpecEventListener.PrecompiledType; +import org.apache.asterix.common.transactions.ITxnIdFactory; import org.apache.asterix.common.config.DatasetConfig.DatasetType; import org.apache.asterix.common.config.DatasetConfig.IndexType; import org.apache.asterix.common.dataflow.ICcApplicationContext; @@ -256,11 +257,11 @@ } private void setupExecutorJob(EntityId entityId, JobSpecification channeljobSpec, IHyracksClientConnection hcc, -DeployedJobSpecEventListener listener) throws Exception { +DeployedJobSpecEventListener listener, ITxnIdFactory txnIdFactory) throws Exception { if (channeljobSpec != null) { DeployedJobSpecId destributedId = hcc.deployJobSpec(channeljobSpec); ScheduledExecutorService ses = DeployedJobService.startRepetitiveDeployedJobSpec(destributedId, hcc, -ChannelJobService.findPeriod(duration), new HashMap<>(), entityId); +ChannelJobService.findPeriod(duration), new HashMap<>(), entityId, txnIdFactory); listener.storeDistributedInfo(destributedId, ses, null, null); } @@ -331,7 +332,7 @@ activeEventHandler.registerListener(listener); } -setupExecutorJob(entityId, channeljobSpec, hcc, listener); +setupExecutorJob(entityId, channeljobSpec, hcc, listener, metadataProvider.getTxnIdFactory()); channel = new Channel(dataverse, channelName.getValue(), subscriptionsTableName, resultsTableName, function, duration, null); diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java index 7ab7f95..7db935a 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java @@ -18,6 +18,12 @@ */ package org.apache.asterix.bad.lang.statement; +import java.io.DataOutput; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.concurrent.ScheduledExecutorService; + import org.apache.asterix.active.DeployedJobService; import org.apache.asterix.active.EntityId; import org.apache.asterix.algebra.extension.IExtensionStatement; @@ -45,7 +51,6 @@ import org.apache.asterix.metadata.declared.MetadataProvider; import org.apache.asterix.om.base.AString; import org.apache.asterix.om.base.IAObject; -import org.apache.asterix.transaction.management.service.transaction.TxnIdFactory; import org.apache.asterix.translator.ConstantHelper; import org.apache.asterix.translator.IRequestParameters; import org.apache.asterix.translator.IStatementExecutor; @@ -56,12 +61,6 @@ import org.apache.hyracks.api.job.DeployedJobSpecId; import org.apache.hyracks.api.job.JobId; import org.apache.hyracks.data.std.util.ArrayBackedValueStorage; - -import java.io.DataOutput; -import java.util.HashMap; -import java.util.List; -import java.util.Map; -import
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Murtadha Hubail has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2361 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2361 to look at the new patch set (#2). Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 --- M asterix-bad/pom.xml M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java 3 files changed, 14 insertions(+), 24 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/61/2361/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2361 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/246/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2361 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/245/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2361 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Michael Blow has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2361 Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2344/ Change-Id: I973c67448d4b34c4521d0abd23c999397e88cf67 --- M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java M asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java 2 files changed, 14 insertions(+), 14 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/61/2361/1 diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java index feaa3ca..53baf6d 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/CreateChannelStatement.java @@ -40,6 +40,7 @@ import org.apache.asterix.bad.metadata.Channel; import org.apache.asterix.bad.metadata.DeployedJobSpecEventListener; import org.apache.asterix.bad.metadata.DeployedJobSpecEventListener.PrecompiledType; +import org.apache.asterix.common.transactions.ITxnIdFactory; import org.apache.asterix.common.config.DatasetConfig.DatasetType; import org.apache.asterix.common.config.DatasetConfig.IndexType; import org.apache.asterix.common.dataflow.ICcApplicationContext; @@ -256,11 +257,11 @@ } private void setupExecutorJob(EntityId entityId, JobSpecification channeljobSpec, IHyracksClientConnection hcc, -DeployedJobSpecEventListener listener) throws Exception { +DeployedJobSpecEventListener listener, ITxnIdFactory txnIdFactory) throws Exception { if (channeljobSpec != null) { DeployedJobSpecId destributedId = hcc.deployJobSpec(channeljobSpec); ScheduledExecutorService ses = DeployedJobService.startRepetitiveDeployedJobSpec(destributedId, hcc, -ChannelJobService.findPeriod(duration), new HashMap<>(), entityId); +ChannelJobService.findPeriod(duration), new HashMap<>(), entityId, txnIdFactory); listener.storeDistributedInfo(destributedId, ses, null, null); } @@ -331,7 +332,7 @@ activeEventHandler.registerListener(listener); } -setupExecutorJob(entityId, channeljobSpec, hcc, listener); +setupExecutorJob(entityId, channeljobSpec, hcc, listener, metadataProvider.getTxnIdFactory()); channel = new Channel(dataverse, channelName.getValue(), subscriptionsTableName, resultsTableName, function, duration, null); diff --git a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java index 7ab7f95..7db935a 100644 --- a/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java +++ b/asterix-bad/src/main/java/org/apache/asterix/bad/lang/statement/ExecuteProcedureStatement.java @@ -18,6 +18,12 @@ */ package org.apache.asterix.bad.lang.statement; +import java.io.DataOutput; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.concurrent.ScheduledExecutorService; + import org.apache.asterix.active.DeployedJobService; import org.apache.asterix.active.EntityId; import org.apache.asterix.algebra.extension.IExtensionStatement; @@ -45,7 +51,6 @@ import org.apache.asterix.metadata.declared.MetadataProvider; import org.apache.asterix.om.base.AString; import org.apache.asterix.om.base.IAObject; -import org.apache.asterix.transaction.management.service.transaction.TxnIdFactory; import org.apache.asterix.translator.ConstantHelper; import org.apache.asterix.translator.IRequestParameters; import org.apache.asterix.translator.IStatementExecutor; @@ -56,12 +61,6 @@ import org.apache.hyracks.api.job.DeployedJobSpecId; import org.apache.hyracks.api.job.JobId; import org.apache.hyracks.data.std.util.ArrayBackedValueStorage; - -import java.io.DataOutput; -import java.util.HashMap; -import java.util.List; -import java.util.Map; -import java.util.concurrent.ScheduledExecutorService; public class ExecuteProcedureStatement implements IExtensionStatement { @@ -133,7 +132,7 @@ if (procedure.getDuration().equals("")) { //Add the Asterix Transaction Id to the map -long newTxId = TxnIdFactory.create().getId(); +long newTxId = metadataProvider.getTxnIdFactory().create().getId(); contextRuntimeVarMap.put(BADConstants.TRANSACTION_ID_PARAMETER_NAME,
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Yingyi Bu has submitted this change and it was merged. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 --- M asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm M asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm 2 files changed, 6 insertions(+), 6 deletions(-) Approvals: Steven Jacobs: Looks good to me, approved Jenkins: Verified diff --git a/asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm b/asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm index dde1ee9..776192d 100644 --- a/asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm +++ b/asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm @@ -1,2 +1,2 @@ -{ "DataverseName": "channels", "DatasetName": "nearbyTweetChannelResults", "DatatypeDataverseName": "Metadata", "DatatypeName": "ChannelResultsType", "DatasetType": "INTERNAL", "GroupName": "DEFAULT_NG_ALL_NODES", "CompactionPolicy": "prefix", "CompactionPolicyProperties": [ { "Name": "max-mergable-component-size", "Value": "1073741824" }, { "Name": "max-tolerance-component-count", "Value": "5" } ], "InternalDetails": { "FileStructure": "BTREE", "PartitioningStrategy": "HASH", "PartitioningKey": [ [ "resultId" ] ], "PrimaryKey": [ [ "resultId" ] ], "Autogenerated": true }, "Hints": {{ }}, "Timestamp": "Mon Sep 12 13:48:16 PDT 2016", "DatasetId": 103, "PendingOp": 0 } -{ "DataverseName": "channels", "DatasetName": "nearbyTweetChannelSubscriptions", "DatatypeDataverseName": "Metadata", "DatatypeName": "ChannelSubscriptionsType", "DatasetType": "INTERNAL", "GroupName": "DEFAULT_NG_ALL_NODES", "CompactionPolicy": "prefix", "CompactionPolicyProperties": [ { "Name": "max-mergable-component-size", "Value": "1073741824" }, { "Name": "max-tolerance-component-count", "Value": "5" } ], "InternalDetails": { "FileStructure": "BTREE", "PartitioningStrategy": "HASH", "PartitioningKey": [ [ "subscriptionId" ] ], "PrimaryKey": [ [ "subscriptionId" ] ], "Autogenerated": true }, "Hints": {{ }}, "Timestamp": "Mon Sep 12 13:48:16 PDT 2016", "DatasetId": 102, "PendingOp": 0 } \ No newline at end of file +{ "DataverseName": "channels", "DatasetName": "nearbyTweetChannelResults", "DatatypeDataverseName": "Metadata", "DatatypeName": "ChannelResultsType", "DatasetType": "INTERNAL", "GroupName": "channels.nearbyTweetChannelResults", "CompactionPolicy": "prefix", "CompactionPolicyProperties": [ { "Name": "max-mergable-component-size", "Value": "1073741824" }, { "Name": "max-tolerance-component-count", "Value": "5" } ], "InternalDetails": { "FileStructure": "BTREE", "PartitioningStrategy": "HASH", "PartitioningKey": [ [ "resultId" ] ], "PrimaryKey": [ [ "resultId" ] ], "Autogenerated": true }, "Hints": {{ }}, "Timestamp": "Mon Sep 12 13:48:16 PDT 2016", "DatasetId": 103, "PendingOp": 0 } +{ "DataverseName": "channels", "DatasetName": "nearbyTweetChannelSubscriptions", "DatatypeDataverseName": "Metadata", "DatatypeName": "ChannelSubscriptionsType", "DatasetType": "INTERNAL", "GroupName": "channels.nearbyTweetChannelSubscriptions", "CompactionPolicy": "prefix", "CompactionPolicyProperties": [ { "Name": "max-mergable-component-size", "Value": "1073741824" }, { "Name": "max-tolerance-component-count", "Value": "5" } ], "InternalDetails": { "FileStructure": "BTREE", "PartitioningStrategy": "HASH", "PartitioningKey": [ [ "subscriptionId" ] ], "PrimaryKey": [ [ "subscriptionId" ] ], "Autogenerated": true }, "Hints": {{ }}, "Timestamp": "Mon Sep 12 13:48:16 PDT 2016", "DatasetId": 102, "PendingOp": 0 } diff --git a/asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm b/asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm index 4002a62..7d160fa 100644 --- a/asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm +++ b/asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm @@ -1,4 +1,4 @@ -{ "DataverseName": "channels", "DatasetName": "nearbyTweetChannel1Results", "DatatypeDataverseName": "Metadata", "DatatypeName": "ChannelResultsType", "DatasetType": "INTERNAL", "GroupName": "DEFAULT_NG_ALL_NODES", "CompactionPolicy": "prefix", "CompactionPolicyProperties": [ { "Name":
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/147/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1802 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1802 to look at the new patch set (#2). Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 --- M asterix-bad/src/test/resources/runtimets/results/channel/create_channel_check_datasets/create_channel_check_datasets.1.adm M asterix-bad/src/test/resources/runtimets/results/channel/drop_channel_check_datasets/drop_channel_check_datasets.1.adm 2 files changed, 6 insertions(+), 6 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/02/1802/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1802 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/146/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1802 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/145/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1802 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...
Jenkins has posted comments on this change. Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/1799 .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/142/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1802 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I43a2d0d1d0e24152acdf428ebbe5c23925f12a53 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-HasComments: No