[GitHub] [activemq-artemis] jdanekrh commented on issue #2792: NO-JIRA: Javadoc for ActiveMQServerMessagePlugin interface

2019-08-15 Thread GitBox
jdanekrh commented on issue #2792: NO-JIRA: Javadoc for 
ActiveMQServerMessagePlugin interface
URL: https://github.com/apache/activemq-artemis/pull/2792#issuecomment-521680441
 
 
   (Does it look reasonable so far, what I wrote? Especially that using 
ThreadLocal is a good idea?)


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [activemq-artemis] jdanekrh commented on issue #2792: NO-JIRA: Javadoc for ActiveMQServerMessagePlugin interface

2019-08-15 Thread GitBox
jdanekrh commented on issue #2792: NO-JIRA: Javadoc for 
ActiveMQServerMessagePlugin interface
URL: https://github.com/apache/activemq-artemis/pull/2792#issuecomment-521549275
 
 
   Ok, I can move it there.
   
   I saw some java classes with quite extensive Javadoc, and I felt that 
explaining how user provided methods are called belongs in a Javadoc.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [activemq-artemis] jdanekrh commented on issue #2792: NO-JIRA: Javadoc for ActiveMQServerMessagePlugin interface

2019-08-12 Thread GitBox
jdanekrh commented on issue #2792: NO-JIRA: Javadoc for 
ActiveMQServerMessagePlugin interface
URL: https://github.com/apache/activemq-artemis/pull/2792#issuecomment-520391046
 
 
   I still need to try this with a broker cluster, to see what exactly happens 
there.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services