[
https://issues.apache.org/jira/browse/GEODE-8435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17180865#comment-17180865
]
ASF GitHub Bot commented on GEODE-8435:
---------------------------------------
onichols-pivotal commented on a change in pull request #5462:
URL: https://github.com/apache/geode/pull/5462#discussion_r473389112
##########
File path:
geode-gfsh/src/main/java/org/apache/geode/management/internal/cli/commands/ConnectCommand.java
##########
@@ -197,6 +201,32 @@ public ResultModel connect(
}
}
+ /**
+ * because remote serialization version was not exposed until 1.12, but we
are compatible back to
+ * 1.10, then any 1.x remote serialization version implies compatibility;
otherwise make some
+ * narrow assumptions about product version numbers known to be associated
with 1.10/1.11 to fill
+ * the gap.
+ *
+ * It seems reasonable to commit to future compatibility with any future
Geode of the same major,
+ * but we should probably draw the line somewhere and not promise that gfsh
1.x will be eternally
+ * compatible with Geode 2.x and later
+ */
+ static boolean isCompatible(String ourSerializationVersion, String
remoteVersion,
Review comment:
I have renamed the method to `shouldConnect` and arranged it as you
suggested to make it more readable, thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> restore gfsh ability to connect if serialization versions match regardless of
> product version
> ---------------------------------------------------------------------------------------------
>
> Key: GEODE-8435
> URL: https://issues.apache.org/jira/browse/GEODE-8435
> Project: Geode
> Issue Type: Bug
> Components: gfsh
> Reporter: Owen Nichols
> Assignee: Owen Nichols
> Priority: Major
> Labels: pull-request-available
>
> This used to work but was broken recently by changes for GEODE-8331.
> Although this does not impact Geode releases, it causes quite a headache for
> developers that don't typically pass a product version in their gradle build
> command.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
