Hello Jianfeng Jia, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1205
to look at the new patch set (#5).
Change subject: Fixes for ASTERIXDB-1636
......................................................................
Fixes for ASTERIXDB-1636
The index of the tuple field for filters from SecondaryIndexOperationsHelper
and AqlMetadataProvider
differed. The one in AqlMetadataProvider was wrong, as it was attempting to
take into account the
presence of a partitioning field in the incoming tuple, which is not there in
the case of an
insert/upsert.
There was also an issue where on merge, for components with a filter page but
no min/max, the merge
would fail. I fixed this by skipping over null entries while getting the
min/max of merging components.
Finally, there was a very silly error in LSMComponentFilterManager which was
causing the filter page
to appear as blank, because the page was being pinned with the wrong argument.
That is also fixed.
Change-Id: Ib4bc413fcda9a5c98ae57f94e1c8a68fe9aacda3
---
M
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/LSMComponentFilterManager.java
M
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/LSMInvertedIndex.java
3 files changed, 22 insertions(+), 18 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/05/1205/5
--
To view, visit https://asterix-gerrit.ics.uci.edu/1205
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib4bc413fcda9a5c98ae57f94e1c8a68fe9aacda3
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
