>From Michael Blow <mb...@apache.org>: Michael Blow has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18267 )
Change subject: [NO ISSUE][HYR] Fix LSMBTreeMergeFailTest on Java 17 ...................................................................... [NO ISSUE][HYR] Fix LSMBTreeMergeFailTest on Java 17 Change-Id: I61f47201c92d6805b50fdc8e7d6f063f31f86cf7 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18267 Reviewed-by: Michael Blow <mb...@apache.org> Reviewed-by: Wail Alkowaileet <wael....@gmail.com> Reviewed-by: Ian Maxon <ima...@uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Tested-by: Ian Maxon <ima...@uci.edu> --- M hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreeMergeFailTest.java 1 file changed, 15 insertions(+), 4 deletions(-) Approvals: Ian Maxon: Looks good to me, approved; Verified Wail Alkowaileet: Looks good to me, approved Michael Blow: Looks good to me, but someone else must approve Jenkins: Verified diff --git a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreeMergeFailTest.java b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreeMergeFailTest.java index bd5d97d..ce16730 100644 --- a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreeMergeFailTest.java +++ b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreeMergeFailTest.java @@ -20,7 +20,6 @@ package org.apache.hyracks.storage.am.lsm.btree; import java.lang.reflect.Field; -import java.lang.reflect.Modifier; import org.apache.hyracks.api.dataflow.value.ISerializerDeserializer; import org.apache.hyracks.api.exceptions.HyracksDataException; @@ -138,9 +137,6 @@ } Field field = MergeOperation.class.getDeclaredField("cursor"); field.setAccessible(true); - Field modifiersField = Field.class.getDeclaredField("modifiers"); - modifiersField.setAccessible(true); - modifiersField.setInt(field, field.getModifiers() & ~Modifier.FINAL); LSMBTreeRangeSearchCursor originalCursor = (LSMBTreeRangeSearchCursor) field.get(operation); field.set(operation, new TestCursor(originalCursor.getOpCtx())); } -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18267 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: trinity Gerrit-Change-Id: I61f47201c92d6805b50fdc8e7d6f063f31f86cf7 Gerrit-Change-Number: 18267 Gerrit-PatchSet: 2 Gerrit-Owner: Michael Blow <mb...@apache.org> Gerrit-Reviewer: Ian Maxon <ima...@uci.edu> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Michael Blow <mb...@apache.org> Gerrit-Reviewer: Murtadha Hubail <mhub...@apache.org> Gerrit-Reviewer: Wail Alkowaileet Gerrit-Reviewer: Wail Alkowaileet <wael....@gmail.com> Gerrit-CC: Anon. E. Moose #1000171 Gerrit-MessageType: merged