clebertsuconic commented on code in PR #6323:
URL: https://github.com/apache/artemis/pull/6323#discussion_r3094906489
##########
tests/compatibility-tests/src/test/java/org/apache/activemq/artemis/tests/compatibility/MultiVersionReplicaTest.java:
##########
@@ -63,13 +64,11 @@ public static Collection getParameters() {
if (getJavaVersion() <= 22) {
// Old 2.x servers fail on JDK23+ without workarounds.
combinations.add(new Object[]{ARTEMIS_2_22_0, SNAPSHOT});
- combinations.add(new Object[]{SNAPSHOT, ARTEMIS_2_22_0});
combinations.add(new Object[]{ARTEMIS_2_17_0, SNAPSHOT});
- combinations.add(new Object[]{SNAPSHOT, ARTEMIS_2_17_0});
+ combinations.add(new Object[]{ARTEMIS_2_33_0, SNAPSHOT});
}
combinations.add(new Object[]{ARTEMIS_2_44_0, SNAPSHOT});
- combinations.add(new Object[]{SNAPSHOT, ARTEMIS_2_44_0});
Review Comment:
version compatibility support is alwasys forward . the backward
compatibility is best effort.. and this case the journal persistence is broken
by V4. the older version won't understand V4.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
