[
https://issues.apache.org/jira/browse/ARTEMIS-2238?focusedWorklogId=190387&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-190387
]
ASF GitHub Bot logged work on ARTEMIS-2238:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 26/Jan/19 10:41
Start Date: 26/Jan/19 10:41
Worklog Time Spent: 10m
Work Description: franz1981 commented on pull request #2520: ARTEMIS-2238
Enhancement to queueQuery on producer
URL: https://github.com/apache/activemq-artemis/pull/2520#discussion_r251195297
##########
File path:
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQSession.java
##########
@@ -397,6 +380,55 @@ public MessageProducer createProducer(final Destination
destination) throws JMSE
}
}
+ void checkDestination(ActiveMQDestination destination) throws JMSException {
Review comment:
I understand it and agree, but depends on the visibility/importance of the
method: this one if overridden really breaks badly pretty much everything.
I won't never use final for private methods, but for package private ie
near-public and important ones, yes.
As I've said is just style comment, I'm already happy with the PR :+1:
Just a side note:
> I have needed quite a few times to override them on test cases
Mockito or similar tools should help on these cases: they basically are gods
amongs mortals when need to deal with language access restrictions
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 190387)
Time Spent: 14h 10m (was: 14h)
> JMS Producer queueQuery does not work when addressName != queueName
> -------------------------------------------------------------------
>
> Key: ARTEMIS-2238
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2238
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Affects Versions: 2.6.4
> Environment: this is a regression created atÂ
> f3ace6afd726dc8e3c1c58c76e3fad3d5cfa357d
> Reporter: clebert suconic
> Priority: Major
> Fix For: 2.7.0, 2.6.5
>
> Time Spent: 14h 10m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
