[GitHub] [geode] dschneider-pivotal commented on pull request #2905: GEODE-6068: undo create jdbc-mapping on destroy jdbc-mapping

2018-11-29 Thread GitHub
In that case we set this attribute to the empty string which means it has no asyncQueueIds. We have a unit test that confirms this. So I don't see any need to check for an empty queue here. The String.join call handles it correctly. [ Full content available at:

[GitHub] [geode] dschneider-pivotal commented on pull request #2905: GEODE-6068: undo create jdbc-mapping on destroy jdbc-mapping

2018-11-29 Thread GitHub
create jdbc-mapping always creates a unique queue for each region. It is 1-to-1. It does not support sharing a single queue on multiple regions. [ Full content available at: https://github.com/apache/geode/pull/2905 ] This message was relayed via gitbox.apache.org for