Ian Maxon has uploaded a new patch set (#2). 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, 11 insertions(+), 5 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/05/1205/2 -- 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: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]>
