clebertsuconic commented on PR #4847:
URL: 
https://github.com/apache/activemq-artemis/pull/4847#issuecomment-1985935031

   I will write a test before merging this...
   
   Although the only thing I will be able to do is to verify the DBOptions gets 
the user and password on the properties.
   
   Otherwise I would need a Database configured just for the test itself. which 
gets a bit fragile.


-- 
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: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to