dschneider-pivotal commented on a change in pull request #7022:
URL: https://github.com/apache/geode/pull/7022#discussion_r734693476



##########
File path: 
geode-core/src/main/java/org/apache/geode/internal/cache/BucketRegion.java
##########
@@ -2497,4 +2500,38 @@ void updateSenderIdMonitor() {
   void checkSameSenderIdsAvailableOnAllNodes() {
     // nothing needed on a bucket region
   }
+
+  @Override
+  public void txApplyPut(Operation putOp, Object key, Object newValue, boolean 
didDestroy,
+      TransactionId transactionId, TXRmtEvent event, EventID eventId, Object 
aCallbackArgument,
+      List<EntryEventImpl> pendingCallbacks, FilterRoutingInfo 
filterRoutingInfo,
+      ClientProxyMembershipID bridgeContext, TXEntryState txEntryState, 
VersionTag versionTag,
+      long tailKey) {
+
+    Object wrappedNewValue;

Review comment:
       Add a system property that defaults to false. If it is true then the old 
behavior will be restored. It is possible an existing customer will be 
dependent on the old "buggy" behavior.




-- 
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]


Reply via email to