[
https://issues.apache.org/jira/browse/GEODE-8870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17272403#comment-17272403
]
ASF GitHub Bot commented on GEODE-8870:
---------------------------------------
pivotal-jbarrett commented on a change in pull request #5947:
URL: https://github.com/apache/geode/pull/5947#discussion_r564814163
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnection.java
##########
@@ -606,38 +605,37 @@ boolean processHandShake() {
return result;
}
} finally {
- if (isTerminated() || !result) {
- return false;
- }
- boolean registerClient = false;
- synchronized (getCleanupProxyIdTable()) {
- MutableInt numRefs = getCleanupProxyIdTable().get(proxyId);
- if (numRefs != null) {
- numRefs.increment();
- } else {
- registerClient = true;
- getCleanupProxyIdTable().put(proxyId, new MutableInt(1));
+ if (!isTerminated() && result) {
Review comment:
It raises a good question. Returning or throwing form a finally block is
strongly discouraged. Given the lack of comments it was hard to infer the
intent. Given your concern that it may have been intentionally discarding
exceptions there is enough reasonable doubt to undo this change. It needs more
investigation to understand the intentions.
----------------------------------------------------------------
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:
us...@infra.apache.org
> Remove obsolete version compatibility code
> ------------------------------------------
>
> Key: GEODE-8870
> URL: https://issues.apache.org/jira/browse/GEODE-8870
> Project: Geode
> Issue Type: Improvement
> Reporter: Jacob Barrett
> Assignee: Jacob Barrett
> Priority: Major
> Labels: pull-request-available
>
> As a followup to GEODE-8837 remove all obsolete backwards compatibility code.
> This ticket will catch all changes to remove the obsolete code.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
