nabarunnag commented on code in PR #7688:
URL: https://github.com/apache/geode/pull/7688#discussion_r893815867
##########
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientUpdateMessageImpl.java:
##########
@@ -1011,7 +1015,7 @@ public void sendTo(DataOutput out) throws IOException {
InternalDataSerializer.writeArrayLength(size, out);
if (size > 0) {
DataSerializer.writeObject(name[0], out);
- DataSerializer.writeObject(op, out);
+ DataSerializer.writeObject(op.id, out);
Review Comment:
Is there a possibility of an NPE here or other places if we call members of
op, if we created CqNameToOpSingleEntry with op value as null. As we can see is
possible in the new code change in HAEventWrapper.
--
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]
