tabish121 commented on code in PR #5113:
URL: https://github.com/apache/activemq-artemis/pull/5113#discussion_r1690400418


##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/Message.java:
##########
@@ -80,6 +80,8 @@ public interface Message {
    // The value is somewhat higher on 64 bit architectures, probably due to 
different alignment
    int memoryOffset = 352;
 
+   int ESTIMATE_DATA_WEIGHT = 2;

Review Comment:
   I generally like to see values like this carry some amount of documentation 
about what they do and why they have the given value just so future maintainers 
don't have to spend time to come up with an educated guess. 



-- 
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]
For further information, visit: https://activemq.apache.org/contact


Reply via email to