clebertsuconic commented on code in PR #6323:
URL: https://github.com/apache/artemis/pull/6323#discussion_r3059088430
##########
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPMessage.java:
##########
@@ -155,7 +155,7 @@ public abstract class AMQPMessage extends RefCountMessage
implements org.apache.
* developing purposes.
*/
public enum MessageDataScanningStatus {
- NOT_SCANNED(0), SCANNED(1);
+ NOT_SCANNED(0), RELOAD_PERSISTENCE(1), SCANNED(2);
Review Comment:
@tabish121 I pushed a commit that completely eliminates the scanning during
reload.
I could remove the RELOAD status (still thinking about it.. it's only needed
for isPersistent now. I could persist it on the persister as well and just have
it NOT SCANNED.
Please check it out..
(these commits will need to be squashed)
One thing I will work on this afternoon is to add a checker for
LargeMessages as well on AMQPReloadFromPersistenceTest
--
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]
