>From Peeyush Gupta <[email protected]>:
Attention is currently required from: Peeyush Gupta, Ali Alsuliman.
Hello Murtadha Hubail, 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/+/17643
to look at the new patch set (#6).
Change subject: [NO ISSUE][TX]: Multiple fixes to atomic statements
......................................................................
[NO ISSUE][TX]: Multiple fixes to atomic statements
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Correctly computes partition count in case of parallel
execution.
- Correctly computes node count in case of a node group
is used.
- Fix for the case when a job prepared message is recieved
after the transaction is aborted.
- Job commit message is sent in commitTransaction method.
This makes sure that the commit messages are sent only
after the job was successfull.
- Fix for the case when job is aborted/cancelled after
flushes are scheduled on atomic datasets but are not
finished.
Change-Id: Ieb029d6273f19fa4bd0e7edfb8897f894c1f5b6e
---
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-3/atomic-statements-3.1.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-3/atomic-statements-3.2.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/atomic-statements/atomic-statements-4/atomic-statements-4.3.adm
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/PrimaryIndexOperationTracker.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-4/atomic-statements-4.2.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-4/atomic-statements-4.3.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/atomic-statements/atomic-statements-3/atomic-statements-3.3.adm
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/GlobalTxManager.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-3/atomic-statements-3.3.query.sqlpp
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/atomic-statements/atomic-statements-4/atomic-statements-4.1.ddl.sqlpp
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/job/listener/NodeJobTracker.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/INodeJobTracker.java
15 files changed, 380 insertions(+), 68 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/43/17643/6
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17643
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: Ieb029d6273f19fa4bd0e7edfb8897f894c1f5b6e
Gerrit-Change-Number: 17643
Gerrit-PatchSet: 6
Gerrit-Owner: Peeyush Gupta <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Peeyush Gupta <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-MessageType: newpatchset
