Hello 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 (#4).
Change subject: Fix for ASTERIXDB-1636
......................................................................
Fix for ASTERIXDB-1636
The index of the tuple field for filters from SecondaryIndexOperationsHelper
and AqlMetadataProvider
seemed to differ. I corrected the one in AQLMetadataProvider as it seemed that
the size was being
used where the length was intended but someone with more expertise in this area
should verify my
assertion.
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, 21 insertions(+), 18 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/05/1205/4
--
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: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
