>From Ian Maxon <[email protected]>:

Attention is currently required from: Ian Maxon, Peeyush Gupta, Preetham 
Poluparthi.

Hello Anon. E. Moose #1000171, Jenkins, Peeyush Gupta, Preetham Poluparthi,

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20835?usp=email

to look at the new patch set (#7).


Change subject: [ASTERIXDB-1888][RT] Use merging connector for pre-sorted load
......................................................................

[ASTERIXDB-1888][RT] Use merging connector for pre-sorted load

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:
The hash merge connector is disabled by default because in certain
circumstances, it can cause a deadlock in large clusters. However,
for the application of LOAD, it can still be useful. It is better
to occasionally have to retry LOAD than it is to suffer with it
writing sort runs for enormous datasets for no good reason.
So, in the case of a LOAD statement with the pre-sorted keyword
given, the proper connector should be used. Furthermore, add a
"false" setting for the hint, to force it off even in this case,
to have an escape hatch if needed.

Change-Id: I646539d88ea2bdc6da4fbf2b6e94e0a61c3125a9
---
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dml/scan-insert-inverted-index-ngram-secondary-index-string-as-primary-key/scan-insert-inverted-index-ngram-secondary-index-string-as-primary-key.2.update.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dml/scan-insert-inverted-index-word-correlated-secondary-index-string-as-primary-key/scan-insert-inverted-index-word-correlated-secondary-index-string-as-primary-key.2.update.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dml/scan-insert-inverted-index-word-secondary-index-string-as-primary-key/scan-insert-inverted-index-word-secondary-index-string-as-primary-key.2.update.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/fuzzyjoin/dblp-string-as-primary-key/dblp-string-as-primary-key.2.update.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/union/union_opt_1/union_opt_1.2.update.sqlpp
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/EnforceStructuralPropertiesRule.java
6 files changed, 21 insertions(+), 9 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/35/20835/7
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20835?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I646539d88ea2bdc6da4fbf2b6e94e0a61c3125a9
Gerrit-Change-Number: 20835
Gerrit-PatchSet: 7
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Peeyush Gupta <[email protected]>
Gerrit-Reviewer: Preetham Poluparthi <[email protected]>
Gerrit-CC: Ali Alsuliman <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Attention: Ian Maxon <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>

Reply via email to