haanhvu edited a comment on pull request #3812:
URL: https://github.com/apache/activemq-artemis/pull/3812#issuecomment-956559907


   @clebertsuconic @gemmellr @jbertram 
   
   I amended the PR as we discussed. Please review. (Sorry I had an important 
certificate exam last week, so I didn't have much time on the PR).
   
   I got `BUILD SUCCESS` with `mvn -Pdev install -Derrorprone`. Not sure if 
it's the same with the Github workflow. Please help activate the workflow.
   
   Also, I feel like I need to add more tests, especially the configuration 
tests, for the overriding behavior. But I'm not sure which test cases should be 
prioritized here. Any advice?


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