[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-18 Thread stanlyDoge
Github user stanlyDoge commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
Aha, thanks for explanation!


---


[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-18 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
I sent a commit to close this PR. definitely a won't fix.


---


[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-18 Thread jbertram
Github user jbertram commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
I think this may be a misunderstanding about how the core client works vs. 
the other protocols. The core client is very simple. It doesn't support 
address/queue auto-create and auto-delete. That functionality is handled by 
each protocol independently. 


---


[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-18 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
this is a won't fix.. close it please!


---


[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-18 Thread stanlyDoge
Github user stanlyDoge commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
While sending AMQP message, addReceiver() is called, which causes creating 
a queue.
While sending openwire message, addConsumer() is called, which also causes 
creating a queue.
But core message is directly sent to queue. It may happen this queue does 
not exist and message is lost.


---


[GitHub] activemq-artemis issue #2020: ARTEMIS-1812 fix for missing (core) messages a...

2018-04-16 Thread gaohoward
Github user gaohoward commented on the issue:

https://github.com/apache/activemq-artemis/pull/2020
  
@stanlyDoge I think that may be fixed somewhere else probably? can you 
provide your test?


---