Young-Seok Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1434: fixed RecoveryIT failure test ......................................................................
ASTERIXDB-1434: fixed RecoveryIT failure test - Fixed Recovery IT failure test. There has been a lurking bug which always set the isPrimaryIndex flag to true incorrectly for secondary LSMBTree indexes. The bug surfaced out from the recent change which removed a Bloom filter from secondary LSMBTree indexes. Change-Id: I85bc3c8445c10da8f2a8c2f334d1eeefb1c0662a Reviewed-on: https://asterix-gerrit.ics.uci.edu/845 Tested-by: Jenkins <[email protected]> Reviewed-by: Michael Blow <[email protected]> --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryBTreeOperationsHelper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Blow: Looks good to me, approved Jenkins: Verified diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryBTreeOperationsHelper.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryBTreeOperationsHelper.java index 6a6538e..c793f59 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryBTreeOperationsHelper.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryBTreeOperationsHelper.java @@ -91,7 +91,7 @@ if (dataset.getDatasetType() == DatasetType.INTERNAL) { //prepare a LocalResourceMetadata which will be stored in NC's local resource repository ILocalResourceMetadata localResourceMetadata = new LSMBTreeLocalResourceMetadata(secondaryTypeTraits, - secondaryComparatorFactories, secondaryBloomFilterKeyFields, true, dataset.getDatasetId(), + secondaryComparatorFactories, secondaryBloomFilterKeyFields, false, dataset.getDatasetId(), mergePolicyFactory, mergePolicyFactoryProperties, filterTypeTraits, filterCmpFactories, secondaryBTreeFields, secondaryFilterFields); localResourceFactoryProvider = new PersistentLocalResourceFactoryProvider(localResourceMetadata, -- To view, visit https://asterix-gerrit.ics.uci.edu/845 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85bc3c8445c10da8f2a8c2f334d1eeefb1c0662a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Young-Seok Kim <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Young-Seok Kim <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
