Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
somandal committed Oct 11, 2022
1 parent 8e8c910 commit df04878
Showing 1 changed file with 8 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1532,7 +1532,8 @@ public void testForwardIndexDisabledOnNewColumnsSV()
4, false, 1, null, false);
Assert.fail("Forward index cannot be disabled for dictionary disabled columns!");
} catch (IllegalStateException e) {
assertEquals(e.getMessage(), "Dictionary disabled columns cannot disable the forward index");
assertEquals(e.getMessage(), "Dictionary disabled column: newForwardIndexDisabledColumnSV cannot disable the "
+ "forward index");
}

constructV1Segment();
Expand All @@ -1546,7 +1547,8 @@ public void testForwardIndexDisabledOnNewColumnsSV()
_newColumnsSchemaWithForwardIndexDisabled, true, true, false, 4, false, 1, null, false);
Assert.fail("Forward index cannot be disabled for dictionary disabled columns!");
} catch (IllegalStateException e) {
assertEquals(e.getMessage(), "Dictionary disabled columns cannot disable the forward index");
assertEquals(e.getMessage(), "Dictionary disabled column: newForwardIndexDisabledColumnSV cannot disable the "
+ "forward index");
}

// Reset the no dictionary columns
Expand Down Expand Up @@ -1600,7 +1602,8 @@ public void testForwardIndexDisabledOnNewColumnsSV()
_newColumnsSchemaWithForwardIndexDisabled, true, true, true, 4, true, 0, null, false);
Assert.fail("Forward index cannot be disabled without enabling the inverted index!");
} catch (IllegalStateException e) {
assertEquals(e.getMessage(), "Inverted index must be enabled for forward index disabled columns");
assertEquals(e.getMessage(), "Inverted index must be enabled for forward index disabled column: "
+ "newForwardIndexDisabledColumnSV");
}

constructV1Segment();
Expand All @@ -1614,7 +1617,8 @@ public void testForwardIndexDisabledOnNewColumnsSV()
NEWLY_ADDED_FORWARD_INDEX_DISABLED_COL_SV, 1, 1, _newColumnsSchemaWithForwardIndexDisabled, true, true, true,
4, true, 0, null, false);
} catch (IllegalStateException e) {
assertEquals(e.getMessage(), "Inverted index must be enabled for forward index disabled columns");
assertEquals(e.getMessage(), "Inverted index must be enabled for forward index disabled column: "
+ "newForwardIndexDisabledColumnSV");
}

_indexLoadingConfig.setForwardIndexDisabledColumns(new HashSet<>());
Expand Down

0 comments on commit df04878

Please sign in to comment.