>From Ali Alsuliman <[email protected]>:
Attention is currently required from: Ali Alsuliman, Hussain Towaileb, Ian
Maxon, Michael Blow, Murtadha Hubail.
Hello Hussain Towaileb, Ian Maxon, Jenkins, Michael Blow, Murtadha Hubail,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21308?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Integration-Tests-1 by Jenkins, Verified-1 by Jenkins
Change subject: [ASTERIXDB-3649][HYR] Ensure reportPartitionConsumed for all
partitions
......................................................................
[ASTERIXDB-3649][HYR] Ensure reportPartitionConsumed for all partitions
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
The following applies to non-async requests.
- Change result set reader to read all locations even empty ones.
Otherwise, if empty locations were left unread, then the result
state in both the NCs and CC will stay around.
- Call reportPartitionConsumed for all partitions for the same reason.
Otherwise, the job record in the CC will stay around.
Ext-ref: MB-71997, MB-69763
Change-Id: I330d0668c8b41d5096d095a6067f8214ca60a4f1
---
M
hyracks-fullstack/hyracks/hyracks-client/src/main/java/org/apache/hyracks/client/result/PartitionResultSetReader.java
M
hyracks-fullstack/hyracks/hyracks-client/src/main/java/org/apache/hyracks/client/result/ResultSetReader.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/result/ResultPartitionManager.java
3 files changed, 11 insertions(+), 16 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/08/21308/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21308?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: lumina
Gerrit-Change-Id: I330d0668c8b41d5096d095a6067f8214ca60a4f1
Gerrit-Change-Number: 21308
Gerrit-PatchSet: 2
Gerrit-Owner: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Ian Maxon <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Michael Blow <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>
