[jira] [Created] (DISPATCH-1026) Router crashing when using sourcePattern/targetPattern with multiple patterns and one of them being user token when trying to open an unauthorized address

2018-06-07 Thread Fernando Giorgetti (JIRA)
Fernando Giorgetti created DISPATCH-1026:


 Summary: Router crashing when using sourcePattern/targetPattern 
with multiple patterns and one of them being user token when trying to open an 
unauthorized address
 Key: DISPATCH-1026
 URL: https://issues.apache.org/jira/browse/DISPATCH-1026
 Project: Qpid Dispatch
  Issue Type: Bug
Reporter: Fernando Giorgetti


When you define a vhost policy with a targetPattern or a sourcePattern using 
multiple addresses and one of them being the ${user} (user token), and if you 
try to connect to an unauthorized address, the router crashes.

Example:

targetPattern: queue/${user}, sample, address

 

If you try to connect a sender to an address, like, "notauthorized" then the 
router crashes with:

qdrouterd: /tmp/qpid-dispatch/src/policy.c:848: 
_qd_policy_approve_link_name_tree: Assertion `0' failed.
Aborted (core dumped)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (QPIDJMS-389) JMS 2.0 Async CompletionListener can be notified in error

2018-06-07 Thread Timothy Bish (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPIDJMS-389?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Timothy Bish resolved QPIDJMS-389.
--
Resolution: Fixed

> JMS 2.0 Async CompletionListener can be notified in error
> -
>
> Key: QPIDJMS-389
> URL: https://issues.apache.org/jira/browse/QPIDJMS-389
> Project: Qpid JMS
>  Issue Type: Bug
>  Components: qpid-jms-client
>Affects Versions: 0.32.0
>Reporter: Timothy Bish
>Assignee: Timothy Bish
>Priority: Major
> Fix For: 0.33.0
>
>
> If two producers are created for a single session and one producer is used to 
> send a message using an CompletionListener and the other producer happens to 
> be remotely closed the sent message CompletionListener will be notified of 
> completion in error if it hadn't already completed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPIDJMS-389) JMS 2.0 Async CompletionListener can be notified in error

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPIDJMS-389?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505417#comment-16505417
 ] 

ASF subversion and git services commented on QPIDJMS-389:
-

Commit 816773722df3abb23d1bca81aa64f353e57c0a02 in qpid-jms's branch 
refs/heads/master from [~tabish121]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-jms.git;h=8167737 ]

QPIDJMS-389 Signal the correct producer completion only

Ensure that if more than one producer exists on a session that the
remote close of one producer does not trigger completions for the other
producer that is waiting for its send to complete.


> JMS 2.0 Async CompletionListener can be notified in error
> -
>
> Key: QPIDJMS-389
> URL: https://issues.apache.org/jira/browse/QPIDJMS-389
> Project: Qpid JMS
>  Issue Type: Bug
>  Components: qpid-jms-client
>Affects Versions: 0.32.0
>Reporter: Timothy Bish
>Assignee: Timothy Bish
>Priority: Major
> Fix For: 0.33.0
>
>
> If two producers are created for a single session and one producer is used to 
> send a message using an CompletionListener and the other producer happens to 
> be remotely closed the sent message CompletionListener will be notified of 
> completion in error if it hadn't already completed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (QPIDJMS-389) JMS 2.0 Async CompletionListener can be notified in error

2018-06-07 Thread Timothy Bish (JIRA)
Timothy Bish created QPIDJMS-389:


 Summary: JMS 2.0 Async CompletionListener can be notified in error
 Key: QPIDJMS-389
 URL: https://issues.apache.org/jira/browse/QPIDJMS-389
 Project: Qpid JMS
  Issue Type: Bug
  Components: qpid-jms-client
Affects Versions: 0.32.0
Reporter: Timothy Bish
Assignee: Timothy Bish
 Fix For: 0.33.0


If two producers are created for a single session and one producer is used to 
send a message using an CompletionListener and the other producer happens to be 
remotely closed the sent message CompletionListener will be notified of 
completion in error if it hadn't already completed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1025) User token not being replaced properly on a vhost policy when defined in the prefix or suffix

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505384#comment-16505384
 ] 

ASF GitHub Bot commented on DISPATCH-1025:
--

GitHub user fgiorgetti opened a pull request:

https://github.com/apache/qpid-dispatch/pull/319

DISPATCH-1025 - Fixes user token replacement used in prefix or suffix



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/fgiorgetti/qpid-dispatch 
fgiorgetti-DISPATCH-1025

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/319.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #319


commit 812fd0014f69c31a1f9e76be48c273e52fa91fb2
Author: Fernando Giorgetti 
Date:   2018-06-07T22:16:19Z

DISPATCH-1025 - Fixes user token replacement used in prefix or suffix




> User token not being replaced properly on a vhost policy when defined in the 
> prefix or suffix
> -
>
> Key: DISPATCH-1025
> URL: https://issues.apache.org/jira/browse/DISPATCH-1025
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Policy Engine
>Reporter: Fernando Giorgetti
>Priority: Major
>
> When a vhost policy is defined with a sources/sourcePattern and/or 
> targets/targetPattern containing a user token (${user}), the token is just 
> being replaced properly when using: sources/targets and only if it is 
> embedded (not being in prefix or suffix).
> This causes policy validation to reject link attachment to the related 
> address.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #319: DISPATCH-1025 - Fixes user token replacemen...

2018-06-07 Thread fgiorgetti
GitHub user fgiorgetti opened a pull request:

https://github.com/apache/qpid-dispatch/pull/319

DISPATCH-1025 - Fixes user token replacement used in prefix or suffix



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/fgiorgetti/qpid-dispatch 
fgiorgetti-DISPATCH-1025

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/qpid-dispatch/pull/319.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #319


commit 812fd0014f69c31a1f9e76be48c273e52fa91fb2
Author: Fernando Giorgetti 
Date:   2018-06-07T22:16:19Z

DISPATCH-1025 - Fixes user token replacement used in prefix or suffix




---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (DISPATCH-1025) User token not being replaced properly on a vhost policy when defined in the prefix or suffix

2018-06-07 Thread Fernando Giorgetti (JIRA)
Fernando Giorgetti created DISPATCH-1025:


 Summary: User token not being replaced properly on a vhost policy 
when defined in the prefix or suffix
 Key: DISPATCH-1025
 URL: https://issues.apache.org/jira/browse/DISPATCH-1025
 Project: Qpid Dispatch
  Issue Type: Bug
  Components: Policy Engine
Reporter: Fernando Giorgetti


When a vhost policy is defined with a sources/sourcePattern and/or 
targets/targetPattern containing a user token (${user}), the token is just 
being replaced properly when using: sources/targets and only if it is embedded 
(not being in prefix or suffix).

This causes policy validation to reject link attachment to the related address.

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-1857) [cpp] no access to AMQP connection offered/desired capabilities

2018-06-07 Thread Alan Conway (JIRA)
Alan Conway created PROTON-1857:
---

 Summary: [cpp] no access to AMQP connection offered/desired 
capabilities
 Key: PROTON-1857
 URL: https://issues.apache.org/jira/browse/PROTON-1857
 Project: Qpid Proton
  Issue Type: Bug
  Components: cpp-binding
Affects Versions: proton-c-0.23.0
Reporter: Alan Conway
Assignee: Alan Conway
 Fix For: proton-c-0.24.0


The C++ binding does not provide access to the offered-capabilities and 
desired-capabilities of a connection, 
[http://docs.oasis-open.org/amqp/core/v1.0/os/amqp-core-transport-v1.0-os.html#type-open]

This looks like an oversight - link capabilities are supported.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-1856) [ruby] auto-accept over-writing user transfer state

2018-06-07 Thread Alan Conway (JIRA)
Alan Conway created PROTON-1856:
---

 Summary: [ruby] auto-accept over-writing user transfer state
 Key: PROTON-1856
 URL: https://issues.apache.org/jira/browse/PROTON-1856
 Project: Qpid Proton
  Issue Type: Bug
  Components: ruby-binding
Affects Versions: proton-c-0.23.0
Reporter: Alan Conway
Assignee: Alan Conway
 Fix For: proton-c-0.24.0


The auto-accept functionality of the MessagingAdapter is incorrectly setting 
the transfer state even if the user had already set it. It is checking for 
"settled?" to decide if it should set state, but this is incorrect since the 
message may be locally but not remotely settled at this point.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-1855) [ruby] copy link terminus state for incoming links

2018-06-07 Thread Alan Conway (JIRA)
Alan Conway created PROTON-1855:
---

 Summary: [ruby] copy link terminus state for incoming links
 Key: PROTON-1855
 URL: https://issues.apache.org/jira/browse/PROTON-1855
 Project: Qpid Proton
  Issue Type: Bug
  Components: ruby-binding
Affects Versions: proton-c-0.23.0
Reporter: Alan Conway
Assignee: Alan Conway
 Fix For: proton-c-0.24.0


For incoming links, the remote terminus state should be copied to the local 
state automatically before the application handlers see the link object. This 
way all relevant link state can be accessed using the simple get/set methods. 
Other bindings do this, it is an oversight that ruby does not.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-941) Router is returning incorrect files from http get requests.

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-941?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505202#comment-16505202
 ] 

ASF subversion and git services commented on DISPATCH-941:
--

Commit 3dbbafc2dc0e2dbd4aa90c1f1509da966360327e in qpid-dispatch's branch 
refs/heads/1.1.x from [~aconway]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=3dbbafc ]

DISPATCH-941: Router is returning incorrect files from http get

Bug in http-libwebsockets code. A partial HTTP handler such as we have needs to
call lws_callback_http_dummy() to fill in default HTTP behaviours. Without that
the LWS library was not properly informed of termination of HTTP transactions
causing confusion if a connection was re-used.

(cherry picked from commit c728d7f41c78ccc8db127b87c11470a5b9bc8c82)


> Router is returning incorrect files from http get requests.
> ---
>
> Key: DISPATCH-941
> URL: https://issues.apache.org/jira/browse/DISPATCH-941
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Routing Engine
>Affects Versions: 1.0.1
>Reporter: Ernest Allen
>Assignee: Alan Conway
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: CaptureApache_angular.PNG, CaptureApache_angular.PNG, 
> CaptureApache_woff2.PNG, CaptureApache_woff2.PNG, CaptureApache_woff2.PNG, 
> CaptureDispatch_angular.PNG, CaptureDispatch_angular.PNG, 
> CaptureDispatch_woff2.PNG
>
>
> When http requests are made from a Windows machine, the http response headers 
> appear to be mixed up or for a different request. For example, when the 
> browser requests the file OpenSans-Regular-webfont.woff2, the response has a 
> content-type of text/javascript and an incorrect content-length. Here is the 
> bad response: !CaptureDispatch_woff2.PNG!
> When the same browser on the same machine requests the same file from Apache 
> tomcat, the response is correct:
> !CaptureApache_woff2.PNG!
> Another example: This time the request is for the javascript file angular.js. 
> Here is the bad response from the router:
> !CaptureDispatch_angular.PNG!
>  
> And here is the correct response from tomcat for the same file:
> !CaptureApache_angular.PNG!
> The incorrect content-types and content-lengths returned by the router lead 
> me to believe that the responses are getting mixed up.
> Possibly important note: Which requests are mixed up varies each time I 
> reload the page. Sometimes files that failed on a previous page load will 
> come down just fine. It doesn't always happen on the same files.
> This happens for all browsers I've tried on Windows. It does not happen for 
> any browser when the request comes from my linux box.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread Ganesh Murthy (JIRA)


 [ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ganesh Murthy resolved DISPATCH-1023.
-
Resolution: Fixed

> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread Ganesh Murthy (JIRA)


 [ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ganesh Murthy updated DISPATCH-1021:

Fix Version/s: 1.1.0

> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread Ganesh Murthy (JIRA)


 [ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ganesh Murthy resolved DISPATCH-1021.
-
Resolution: Fixed

> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505061#comment-16505061
 ] 

ASF subversion and git services commented on DISPATCH-1023:
---

Commit 478a98688afbaa1c62abbc7a53775e1fae0dfb15 in qpid-dispatch's branch 
refs/heads/1.1.x from [~ganeshmurthy]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=478a986 ]

DISPATCH-1023 - Added a multicast bit on the qdr_delivery_t object to 
use in places where owning_addr is not available. Also added code to free multi 
frame non-multicast presettled messages. This closes #318

(cherry picked from commit e96c7ed5083d4563f8b82b852d3367d59da9)


> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505060#comment-16505060
 ] 

ASF subversion and git services commented on DISPATCH-1021:
---

Commit edfa1ccc5859ea3567b641e1ced8fabda9055d65 in qpid-dispatch's branch 
refs/heads/1.1.x from [~ganeshmurthy]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=edfa1cc ]

DISPATCH-1021 - Introduced a more flag in the action so every action can 
accurately know if there is more data to come in the message. This closes #316.

(cherry picked from commit c1e0ceeecc9348c7633022681a7ac19e6fd0f1b8)


> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505054#comment-16505054
 ] 

ASF GitHub Bot commented on DISPATCH-1023:
--

Github user asfgit closed the pull request at:

https://github.com/apache/qpid-dispatch/pull/318


> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505053#comment-16505053
 ] 

ASF subversion and git services commented on DISPATCH-1023:
---

Commit e96c7ed5083d4563f8b82b852d3367d59da9 in qpid-dispatch's branch 
refs/heads/master from [~ganeshmurthy]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=e96c7ed ]

DISPATCH-1023 - Added a multicast bit on the qdr_delivery_t object to 
use in places where owning_addr is not available. Also added code to free multi 
frame non-multicast presettled messages. This closes #318


> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #318: DISPATCH-1023 - Added a multicast bit on th...

2018-06-07 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/qpid-dispatch/pull/318


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505035#comment-16505035
 ] 

ASF subversion and git services commented on DISPATCH-1021:
---

Commit c1e0ceeecc9348c7633022681a7ac19e6fd0f1b8 in qpid-dispatch's branch 
refs/heads/master from [~ganeshmurthy]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=c1e0cee ]

DISPATCH-1021 - Introduced a more flag in the action so every action can 
accurately know if there is more data to come in the message. This closes #316.


> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505036#comment-16505036
 ] 

ASF GitHub Bot commented on DISPATCH-1021:
--

Github user asfgit closed the pull request at:

https://github.com/apache/qpid-dispatch/pull/316


> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #316: DISPATCH-1021 - Introduced a more flag in t...

2018-06-07 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/qpid-dispatch/pull/316


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505011#comment-16505011
 ] 

ASF GitHub Bot commented on DISPATCH-1021:
--

Github user ganeshmurthy commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/316#discussion_r193836829
  
--- Diff: src/router_core/transfer.c ---
@@ -1237,7 +1246,7 @@ void qdr_drain_inbound_undelivered_CT(qdr_core_t 
*core, qdr_link_t *link, qdr_ad
 qdr_delivery_t *dlv = DEQ_HEAD(deliveries);
 while (dlv) {
 DEQ_REMOVE_HEAD(deliveries);
-qdr_link_forward_CT(core, link, dlv, addr);
+qdr_link_forward_CT(core, link, dlv, addr, 0);
--- End diff --

will do. thanks.


> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #316: DISPATCH-1021 - Introduced a more flag in t...

2018-06-07 Thread ganeshmurthy
Github user ganeshmurthy commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/316#discussion_r193836829
  
--- Diff: src/router_core/transfer.c ---
@@ -1237,7 +1246,7 @@ void qdr_drain_inbound_undelivered_CT(qdr_core_t 
*core, qdr_link_t *link, qdr_ad
 qdr_delivery_t *dlv = DEQ_HEAD(deliveries);
 while (dlv) {
 DEQ_REMOVE_HEAD(deliveries);
-qdr_link_forward_CT(core, link, dlv, addr);
+qdr_link_forward_CT(core, link, dlv, addr, 0);
--- End diff --

will do. thanks.


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1021) Router core dumps when 3 senders and 3 receivers are sending/receiving very large presettled messages to a multicast address

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505009#comment-16505009
 ] 

ASF GitHub Bot commented on DISPATCH-1021:
--

Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/316#discussion_r193835316
  
--- Diff: src/router_core/transfer.c ---
@@ -1237,7 +1246,7 @@ void qdr_drain_inbound_undelivered_CT(qdr_core_t 
*core, qdr_link_t *link, qdr_ad
 qdr_delivery_t *dlv = DEQ_HEAD(deliveries);
 while (dlv) {
 DEQ_REMOVE_HEAD(deliveries);
-qdr_link_forward_CT(core, link, dlv, addr);
+qdr_link_forward_CT(core, link, dlv, addr, 0);
--- End diff --

You should use false for a boolean literal, not 0.


> Router core dumps when 3 senders and 3 receivers are sending/receiving very 
> large presettled messages to a multicast address 
> -
>
> Key: DISPATCH-1021
> URL: https://issues.apache.org/jira/browse/DISPATCH-1021
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Steps to reproduce
>  * Modify simple_send.py to send large messages (see attachment)
>  * Start 2 routers with attached config files
>  * Attach all senders to one router and all receivers to the other router
>  * The router will crash on the line - assert(in_dlv->where == 
> QDR_DELIVERY_IN_SETTLED;



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #316: DISPATCH-1021 - Introduced a more flag in t...

2018-06-07 Thread ted-ross
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/316#discussion_r193835316
  
--- Diff: src/router_core/transfer.c ---
@@ -1237,7 +1246,7 @@ void qdr_drain_inbound_undelivered_CT(qdr_core_t 
*core, qdr_link_t *link, qdr_ad
 qdr_delivery_t *dlv = DEQ_HEAD(deliveries);
 while (dlv) {
 DEQ_REMOVE_HEAD(deliveries);
-qdr_link_forward_CT(core, link, dlv, addr);
+qdr_link_forward_CT(core, link, dlv, addr, 0);
--- End diff --

You should use false for a boolean literal, not 0.


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505004#comment-16505004
 ] 

ASF GitHub Bot commented on DISPATCH-1023:
--

Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/318#discussion_r193834441
  
--- Diff: src/router_core/transfer.c ---
@@ -1162,7 +1164,7 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, 
qdr_action_t *action, bool
 }
 
 // This is a multicast delivery
-if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr)) {
+if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr) || 
in_dlv->multicast || in_dlv->settled) {
--- End diff --

If the latter is handling for multi-frame unicasts, then the comment above 
is no longer accurate.


> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8206) [linearstore] Deadlock possible in InactivityFireEvent if fire() is called at the same time as cancel()

2018-06-07 Thread Kim van der Riet (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505005#comment-16505005
 ] 

Kim van der Riet commented on QPID-8206:


I cannot reproduce using this reproducer on my hardware.

*But* if I add a sleep of 10ms into {{InactivityFireEvent::cancel()}} just 
after taking the lock, then the deadlock occurs quite quickly for me - after a 
minute or so of running Pavel's reproducer.
{noformat}
diff --git a/src/qpid/linearstore/JournalImpl.cpp 
b/src/qpid/linearstore/JournalImpl.cpp
index fe919fbd3..e512e9a05 100644
--- a/src/qpid/linearstore/JournalImpl.cpp
+++ b/src/qpid/linearstore/JournalImpl.cpp
@@ -82,6 +82,7 @@ void InactivityFireEvent::fire() {

 void InactivityFireEvent::cancel() {
 ::qpid::sys::Mutex::ScopedLock sl(_ifeStateLock);
+::usleep(10 * 1000); // <-- *** DEBUG ***
 ::qpid::sys::TimerTask::cancel();
 _state = CANCELLED;
 }
{noformat}
After making the following fix, but leaving the sleep in place:
{noformat}
diff --git a/src/qpid/linearstore/JournalImpl.cpp 
b/src/qpid/linearstore/JournalImpl.cpp
index fe919fbd3..55f33bb80 100644
--- a/src/qpid/linearstore/JournalImpl.cpp
+++ b/src/qpid/linearstore/JournalImpl.cpp
@@ -81,9 +81,12 @@ void InactivityFireEvent::fire() {
 }

 void InactivityFireEvent::cancel() {
-::qpid::sys::Mutex::ScopedLock sl(_ifeStateLock);
 ::qpid::sys::TimerTask::cancel();
-_state = CANCELLED;
+{
+::qpid::sys::Mutex::ScopedLock sl(_ifeStateLock);
+::usleep(10 * 1000); // <-- *** DEBUG ***
+_state = CANCELLED;
+}
 }

 GetEventsFireEvent::GetEventsFireEvent(JournalImpl* p,
{noformat}
I cannot get the reproducer to fail. I am hopeful that this will make a valid 
fix. I will continue to test for a while.

In essence, the fix allows the underlying timer to *first* cancel (which 
includes waiting for any flush fires that may have occurred on another thread) 
before taking the lock to change the local state to CANCELLED.

To be sure we are not introducing a regression, we should probably also check 
the reproducers for QPID-7975.

> [linearstore] Deadlock possible in InactivityFireEvent if fire() is called at 
> the same time as cancel()
> ---
>
> Key: QPID-8206
> URL: https://issues.apache.org/jira/browse/QPID-8206
> Project: Qpid
>  Issue Type: Bug
>  Components: C++ Broker
>Reporter: Kim van der Riet
>Assignee: Kim van der Riet
>Priority: Major
>
> A deadlock has been observed in InactivityFireEvent if 
> {{InactivityFireEvent::fire()}} triggered by the timer occurs at almost the 
> same time as {{InactivityFireEvent::cancel()}} on another thread, and which 
> occurs if the queue is deleted.
> The mutex {{InactivityFireEvent::_ifeStateLock}} becomes deadlocked if the 
> thread calling {{cancel()}} obtains the lock, but a fire event is imminent. 
> The {{fire()}} call will then be blocked on this mutex. However, cancel 
> cannot complete until fire competes owing to the sys::Time Monitor which 
> waits for all fires to complete before allowing the cancel to occur.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #318: DISPATCH-1023 - Added a multicast bit on th...

2018-06-07 Thread ted-ross
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/318#discussion_r193834441
  
--- Diff: src/router_core/transfer.c ---
@@ -1162,7 +1164,7 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, 
qdr_action_t *action, bool
 }
 
 // This is a multicast delivery
-if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr)) {
+if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr) || 
in_dlv->multicast || in_dlv->settled) {
--- End diff --

If the latter is handling for multi-frame unicasts, then the comment above 
is no longer accurate.


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (DISPATCH-1023) qdr_deliovery_t objects leak in two router presettled case

2018-06-07 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/DISPATCH-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16505003#comment-16505003
 ] 

ASF GitHub Bot commented on DISPATCH-1023:
--

Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/318#discussion_r193833973
  
--- Diff: src/router_core/transfer.c ---
@@ -1162,7 +1164,7 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, 
qdr_action_t *action, bool
 }
 
 // This is a multicast delivery
-if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr)) {
+if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr) || 
in_dlv->multicast || in_dlv->settled) {
--- End diff --

Why did you keep the call to qdr_is_addr_treatment_multicast here?  Isn't 
it always true that the in_dlv->multicast flag will be initialized 
appropriately in qdr_link_forward_CT?

Also, why did you add the in_dlv->settled condition?  That is new/changed 
functionality.


> qdr_deliovery_t objects leak in two router presettled case
> --
>
> Key: DISPATCH-1023
> URL: https://issues.apache.org/jira/browse/DISPATCH-1023
> Project: Qpid Dispatch
>  Issue Type: Bug
>  Components: Container
>Affects Versions: 1.0.1
>Reporter: Ganesh Murthy
>Assignee: Ganesh Murthy
>Priority: Major
> Fix For: 1.1.0
>
> Attachments: router1-qdrouterd.conf, router2-qdrouterd.conf, 
> simple_send_large.py
>
>
> Start routers with the attached config files.
> Put a receiver on one router and a receiver on the other
> Send/receive about 10,000 large presettled messages
> Notice that the count of qdr_delivery_t objects in the output of qdstat -m 
> keeps steadily increasing
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[GitHub] qpid-dispatch pull request #318: DISPATCH-1023 - Added a multicast bit on th...

2018-06-07 Thread ted-ross
Github user ted-ross commented on a diff in the pull request:

https://github.com/apache/qpid-dispatch/pull/318#discussion_r193833973
  
--- Diff: src/router_core/transfer.c ---
@@ -1162,7 +1164,7 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, 
qdr_action_t *action, bool
 }
 
 // This is a multicast delivery
-if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr)) {
+if (qdr_is_addr_treatment_multicast(in_dlv->link->owning_addr) || 
in_dlv->multicast || in_dlv->settled) {
--- End diff --

Why did you keep the call to qdr_is_addr_treatment_multicast here?  Isn't 
it always true that the in_dlv->multicast flag will be initialized 
appropriately in qdr_link_forward_CT?

Also, why did you add the in_dlv->settled condition?  That is new/changed 
functionality.


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8206) [linearstore] Deadlock possible in InactivityFireEvent if fire() is called at the same time as cancel()

2018-06-07 Thread Kim van der Riet (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504958#comment-16504958
 ] 

Kim van der Riet commented on QPID-8206:


Pavel Moravec has found a reproducer:

0) {{rm -rf /var/lib/qpidd/.qpidd # just to start with absolutely clean table}}


1) set

{{journal-flush-timeout=50ms}}

to qpidd.conf - the lower the better. I reproduce the deadlock always within a 
minute or two with this value, but it took ~10 minutes with default 500ms (I 
think).


2) run this script that sporadically sends and receives a durable message to a 
queue that is time to time deleted - and it does so concurrently for 50 queues:
{noformat}
for i in $(seq 1 50); do
  while true; do
echo "$(date): loop $i"
qpid-send -a "DurableTest-${i}; {create:always, node:{type:queue, 
durable:true}}" -m 1 --content-size=1100 --durable=yes
qpid-receive -a DurableTest-${i} --print-content=no 2> /dev/null
sleep 0.2
  done &
done

for i in $(seq 1 50); do
  while true; do
echo "$(date): deleting queue $i"
qpid-config del queue DurableTest-${i} --force
sleep $((RANDOM%5)).$((RANDOM%10))
  done &
done
{noformat}

3) check if qpidd responds, via e.g.:

{{while true; do date; qpid-stat -g; sleep 5; done}}

4) within a minute or so, deadlock comes to you and your broker.

> [linearstore] Deadlock possible in InactivityFireEvent if fire() is called at 
> the same time as cancel()
> ---
>
> Key: QPID-8206
> URL: https://issues.apache.org/jira/browse/QPID-8206
> Project: Qpid
>  Issue Type: Bug
>  Components: C++ Broker
>Reporter: Kim van der Riet
>Assignee: Kim van der Riet
>Priority: Major
>
> A deadlock has been observed in InactivityFireEvent if 
> {{InactivityFireEvent::fire()}} triggered by the timer occurs at almost the 
> same time as {{InactivityFireEvent::cancel()}} on another thread, and which 
> occurs if the queue is deleted.
> The mutex {{InactivityFireEvent::_ifeStateLock}} becomes deadlocked if the 
> thread calling {{cancel()}} obtains the lock, but a fire event is imminent. 
> The {{fire()}} call will then be blocked on this mutex. However, cancel 
> cannot complete until fire competes owing to the sys::Time Monitor which 
> waits for all fires to complete before allowing the cancel to occur.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (QPID-8206) [linearstore] Deadlock possible in InactivityFireEvent if fire() is called at the same time as cancel()

2018-06-07 Thread Kim van der Riet (JIRA)
Kim van der Riet created QPID-8206:
--

 Summary: [linearstore] Deadlock possible in InactivityFireEvent if 
fire() is called at the same time as cancel()
 Key: QPID-8206
 URL: https://issues.apache.org/jira/browse/QPID-8206
 Project: Qpid
  Issue Type: Bug
  Components: C++ Broker
Reporter: Kim van der Riet
Assignee: Kim van der Riet


A deadlock has been observed in InactivityFireEvent if 
{{InactivityFireEvent::fire()}} triggered by the timer occurs at almost the 
same time as {{InactivityFireEvent::cancel()}} on another thread, and which 
occurs if the queue is deleted.

The mutex {{InactivityFireEvent::_ifeStateLock}} becomes deadlocked if the 
thread calling {{cancel()}} obtains the lock, but a fire event is imminent. The 
{{fire()}} call will then be blocked on this mutex. However, cancel cannot 
complete until fire competes owing to the sys::Time Monitor which waits for all 
fires to complete before allowing the cancel to occur.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPIDJMS-386) failover provider prevents handling of remote resource closure from completing correctly

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPIDJMS-386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504867#comment-16504867
 ] 

ASF subversion and git services commented on QPIDJMS-386:
-

Commit fe4aa6ed9f9be25ac74c34f07c87859d64b3c73d in qpid-jms's branch 
refs/heads/master from [~gemmellr]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-jms.git;h=fe4aa6e ]

QPIDJMS-386: add some tweaks for the tests


> failover provider prevents handling of remote resource closure from 
> completing correctly
> 
>
> Key: QPIDJMS-386
> URL: https://issues.apache.org/jira/browse/QPIDJMS-386
> Project: Qpid JMS
>  Issue Type: Bug
>  Components: qpid-jms-client
>Affects Versions: 0.32.0
> Environment:  
>Reporter: Robbie Gemmell
>Assignee: Timothy Bish
>Priority: Major
> Fix For: 0.33.0
>
>
> QPIDJMS-376 added functionality to fire the connection ExceptionListener if a 
> MessageConsumer with a MessageListener is remotely closed, since the lack of 
> future message deliveries would mean the application has no further 
> interaction with the consumer to observe the issue. It has since been 
> identified that this doesn't occur when using the built in failover provider, 
> which is preventing the process of handling the remote consumer resource 
> closure from completing fully.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy updated QPID-8204:
-
Priority: Minor  (was: Major)

> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Minor
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (QPID-8202) [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy resolved QPID-8202.
--
Resolution: Fixed

Resolving JIRA as changes have been reviewed and merged into 7.0.x branch

> [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded 
> from store for every content chunk sent to the client
> ---
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple frames. 
> Currently, the flowed to disk message is repeatedly loaded from store and 
> evicted from memory on sending every content chunk. As result, the message 
> consumption performance is very slow. Only AMQP  protocols 0-8..0-91 are 
> affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy resolved QPID-8204.
--
Resolution: Fixed

Resolving JIRA as changes have been reviewed and merged into 7.0.x branch

> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8202) [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504679#comment-16504679
 ] 

ASF subversion and git services commented on QPID-8202:
---

Commit daf346efcd5ca2652f49077c622a3ffccab03049 in qpid-broker-j's branch 
refs/heads/7.0.x from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=daf346e ]

QPID-8202: [Broker-J] Address review comments from Keith Wall

(cherry picked from commit 9d08c765155f4d70e53f972f1a63b4a7326f)


> [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded 
> from store for every content chunk sent to the client
> ---
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple frames. 
> Currently, the flowed to disk message is repeatedly loaded from store and 
> evicted from memory on sending every content chunk. As result, the message 
> consumption performance is very slow. Only AMQP  protocols 0-8..0-91 are 
> affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504678#comment-16504678
 ] 

ASF subversion and git services commented on QPID-8204:
---

Commit 0df1680d9d008d306c5f5e6319e1c5f618f4b5cb in qpid-broker-j's branch 
refs/heads/7.0.x from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=0df1680 ]

QPID-8204: [Broker-J] Rename new statistics into 'inboundMessageHighWatermark' 
and remove extra switch enabling the statistics evaluation

(cherry picked from commit 0b63b074e0e61e026caab0122aa4fa404aec25ff)


> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8202) [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Keith Wall (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504619#comment-16504619
 ] 

Keith Wall commented on QPID-8202:
--

Changes look reasonable to me.

> [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded 
> from store for every content chunk sent to the client
> ---
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple frames. 
> Currently, the flowed to disk message is repeatedly loaded from store and 
> evicted from memory on sending every content chunk. As result, the message 
> consumption performance is very slow. Only AMQP  protocols 0-8..0-91 are 
> affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread Keith Wall (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504610#comment-16504610
 ] 

Keith Wall commented on QPID-8204:
--

Changes look reasonable to me.

> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-8203) [Broker-J][AMQP 0-8...0-91] Improve maximum message size check

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8203?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy updated QPID-8203:
-
Summary: [Broker-J][AMQP 0-8...0-91] Improve maximum message size check  
(was: [Broker-J][AMQP 0-9] Improve maximum message size check)

> [Broker-J][AMQP 0-8...0-91] Improve maximum message size check
> --
>
> Key: QPID-8203
> URL: https://issues.apache.org/jira/browse/QPID-8203
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.0.3, qpid-java-broker-7.0.2, 
> qpid-java-broker-7.0.0, qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> Maximum message size check needs to be improved



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (QPID-8205) [Broker-J] Release Qpid Broker-J 7.0.5

2018-06-07 Thread Alex Rudyy (JIRA)
Alex Rudyy created QPID-8205:


 Summary: [Broker-J] Release Qpid Broker-J 7.0.5
 Key: QPID-8205
 URL: https://issues.apache.org/jira/browse/QPID-8205
 Project: Qpid
  Issue Type: Task
  Components: Broker-J
Reporter: Alex Rudyy
 Fix For: qpid-java-broker-7.0.5


Release Qpid Broker-J following instructions provided at 
[https://cwiki.apache.org/confluence/display/qpid/Releasing+Qpid+Broker-J]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Assigned] (QPID-8205) [Broker-J] Release Qpid Broker-J 7.0.5

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy reassigned QPID-8205:


Assignee: Alex Rudyy

> [Broker-J] Release Qpid Broker-J 7.0.5
> --
>
> Key: QPID-8205
> URL: https://issues.apache.org/jira/browse/QPID-8205
> Project: Qpid
>  Issue Type: Task
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.0.5
>
>
> Release Qpid Broker-J following instructions provided at 
> [https://cwiki.apache.org/confluence/display/qpid/Releasing+Qpid+Broker-J]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-8202) [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy updated QPID-8202:
-
Summary: [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be 
re-loaded from store for every content chunk sent to the client  (was: 
[Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store 
for every content chunk sent to the client)

> [Broker-J][AMQP 0-8...0-91] Large flowed to disk message can be re-loaded 
> from store for every content chunk sent to the client
> ---
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple frames. 
> Currently, the flowed to disk message is repeatedly loaded from store and 
> evicted from memory on sending every content chunk. As result, the message 
> consumption performance is very slow. Only AMQP  protocols 0-8..0-91 are 
> affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-8202) [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy updated QPID-8202:
-
Description: When large message cannot fit in one frame it is sent in 
multiple frames. Currently, the flowed to disk message is repeatedly loaded 
from store and evicted from memory on sending every content chunk. As result, 
the message consumption performance is very slow. Only AMQP  protocols 
0-8..0-91 are affected by the issue  (was: When large message cannot fit in one 
frame it is sent in multiple blocks. Currently, the flowed to disk message is 
reloaded from store and evicted from memory on sending every block. As result, 
the message consumption performance is very slow. Only AMQP  protocols 
0-8..0-91 are affected by the issue)

> [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store 
> for every content chunk sent to the client
> 
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple frames. 
> Currently, the flowed to disk message is repeatedly loaded from store and 
> evicted from memory on sending every content chunk. As result, the message 
> consumption performance is very slow. Only AMQP  protocols 0-8..0-91 are 
> affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Updated] (QPID-8202) [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Alex Rudyy (JIRA)


 [ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Rudyy updated QPID-8202:
-
Affects Version/s: qpid-java-6.1.6
   qpid-java-6.1
   qpid-java-6.1.1
   qpid-java-6.1.2
   qpid-java-6.1.3
   qpid-java-6.1.4
   qpid-java-6.1.5

> [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store 
> for every content chunk sent to the client
> 
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-6.1.6, qpid-java-broker-7.0.3, 
> qpid-java-broker-7.0.2, qpid-java-6.1, qpid-java-6.1.1, qpid-java-6.1.2, 
> qpid-java-6.1.3, qpid-java-6.1.4, qpid-java-broker-7.0.0, qpid-java-6.1.5, 
> qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple blocks. 
> Currently, the flowed to disk message is reloaded from store and evicted from 
> memory on sending every block. As result, the message consumption performance 
> is very slow. Only AMQP  protocols 0-8..0-91 are affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8202) [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread Alex Rudyy (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504538#comment-16504538
 ] 

Alex Rudyy commented on QPID-8202:
--

Keith,
I committed changes 
[9d08c765155f4d70e53f972f1a63b4a7326f|https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=9d08c76]
 addressing your review comments. I decided to continue using 
MessageContentSourceBody as ContentBody duplicates the buffer before sending. 
You are right, 6.1 is affected as well. 

> [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store 
> for every content chunk sent to the client
> 
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.0.3, qpid-java-broker-7.0.2, 
> qpid-java-broker-7.0.0, qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple blocks. 
> Currently, the flowed to disk message is reloaded from store and evicted from 
> memory on sending every block. As result, the message consumption performance 
> is very slow. Only AMQP  protocols 0-8..0-91 are affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8202) [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store for every content chunk sent to the client

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504527#comment-16504527
 ] 

ASF subversion and git services commented on QPID-8202:
---

Commit 9d08c765155f4d70e53f972f1a63b4a7326f in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=9d08c76 ]

QPID-8202: [Broker-J] Address review comments from Keith Wall


> [Broker-J][AMQP 0-9] Large flowed to disk message can be re-loaded from store 
> for every content chunk sent to the client
> 
>
> Key: QPID-8202
> URL: https://issues.apache.org/jira/browse/QPID-8202
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.0.3, qpid-java-broker-7.0.2, 
> qpid-java-broker-7.0.0, qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> When large message cannot fit in one frame it is sent in multiple blocks. 
> Currently, the flowed to disk message is reloaded from store and evicted from 
> memory on sending every block. As result, the message consumption performance 
> is very slow. Only AMQP  protocols 0-8..0-91 are affected by the issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-1846) [proton-c] Message decode fails with PN_OUT_OF_MEMORY if there are large lists in the message

2018-06-07 Thread Marcel Meulemans (JIRA)


[ 
https://issues.apache.org/jira/browse/PROTON-1846?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504462#comment-16504462
 ] 

Marcel Meulemans commented on PROTON-1846:
--

I tried the diff above and and the {{uint32_t}} seems to introduce some 
unwanted side effects due to the with the singed/unsigned "magic" in 
{{pn_data_point/pn_data_restore}} (did look into the details, just that this 
branch, 
[https://github.com/apache/qpid-proton/blob/master/c/src/core/codec.c#L1177] 
isn't hit when it should be). Using {{typedef int32_t pni_nid_t;}} and fixing 
PNI_NID_MAX accordingly did worked for me.

> [proton-c]  Message decode fails with PN_OUT_OF_MEMORY if there are large 
> lists in the message
> --
>
> Key: PROTON-1846
> URL: https://issues.apache.org/jira/browse/PROTON-1846
> Project: Qpid Proton
>  Issue Type: Bug
>  Components: proton-c
>Affects Versions: proton-c-0.22.0
>Reporter: Ganesh Murthy
>Priority: Major
> Attachments: send_large_structured_body.js
>
>
> Steps to reproduce -
>  
>  # Start the Qpid Dispatch router
>  # Run the following script that creates a bunch of addresses
>  # for i in `seq 1 6546`; do echo 
> "\{\"prefix\":\"address-$i\",\"distribution\":\"balanced\"}" | qdmanage 
> CREATE --type=org.apache.qpid.dispatch.router.config.address --name 
> address-$i --stdin; done
>  # now run qdmanage QUERY --type=address
>  # You will receive a Data error (-10)
> The following diff seems to fix the issue
> diff --git a/c/src/core/data.h b/c/src/core/data.h
> index 94dc7d67..f4320e2a 100644
> --- a/c/src/core/data.h
> +++ b/c/src/core/data.h
> @@ -27,7 +27,7 @@
>  #include "decoder.h"
>  #include "encoder.h"
>  
> -typedef uint16_t pni_nid_t;
> +typedef uint32_t pni_nid_t;
>  #define PNI_NID_MAX ((pni_nid_t)-1)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread Alex Rudyy (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504446#comment-16504446
 ] 

Alex Rudyy commented on QPID-8204:
--

Kieth,

I addressed you review comments in commit 
[0b63b074e0e61e026caab0122aa4fa404aec25ff|https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=0b63b07].

> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504441#comment-16504441
 ] 

ASF subversion and git services commented on QPID-8204:
---

Commit 0b63b074e0e61e026caab0122aa4fa404aec25ff in qpid-broker-j's branch 
refs/heads/master from [~alex.rufous]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-broker-j.git;h=0b63b07 ]

QPID-8204: [Broker-J] Rename new statistics into 'inboundMessageHighWatermark' 
and remove extra switch enabling the statistics evaluation


> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Comment Edited] (QPID-8203) [Broker-J][AMQP 0-9] Improve maximum message size check

2018-06-07 Thread Keith Wall (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503972#comment-16503972
 ] 

Keith Wall edited comment on QPID-8203 at 6/7/18 7:43 AM:
--

Production code change (master.7.0.x) looked reasonable.   The new protocol 
test file didn't follow the naming conventions demanded by the configuration of 
the maven-surefire-plugin, so wasn't being run when invoked from Maven.  I 
fixed that and added a 0-10 protocol test. I noted that an equivalent AMQP 1.0 
test already existed.


was (Author: k-wall):
Production code change (master.7.0.x) looked reasonable.   The new protocol 
test file didn't follow the naming conventions demanded by the configuration of 
the maven-surefire-plugin, so wasn't being run when invoked from Maven.  I 
fixed that and added a 0-10 protocol test. I noted that a AMQP 1.0 already 
existed.

> [Broker-J][AMQP 0-9] Improve maximum message size check
> ---
>
> Key: QPID-8203
> URL: https://issues.apache.org/jira/browse/QPID-8203
> Project: Qpid
>  Issue Type: Bug
>  Components: Broker-J
>Affects Versions: qpid-java-broker-7.0.3, qpid-java-broker-7.0.2, 
> qpid-java-broker-7.0.0, qpid-java-broker-7.0.1, qpid-java-broker-7.0.4
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Critical
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
>
> Maximum message size check needs to be improved



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Comment Edited] (QPID-8204) [Broker-J] Add statistics to report the maximum size of incoming messages

2018-06-07 Thread Keith Wall (JIRA)


[ 
https://issues.apache.org/jira/browse/QPID-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504040#comment-16504040
 ] 

Keith Wall edited comment on QPID-8204 at 6/7/18 7:42 AM:
--

Is the {{maxMessageSizeStatisticsEnabled}} switch really justifiable?  I prefer 
to avoid a proliferation of switches because they just bewilder.  My guess 
would be the additional CAS would have no noticeable effect on performance.


was (Author: k-wall):
Is the {{maxMessageSizeStatisticsEnabled}} switch really justifiable?  I prefer 
to avoid a proliferation of switches because they just bewilder.  I guess would 
be the additional CAS would have no noticeable effect on performance.

> [Broker-J] Add statistics to report the maximum size of incoming messages
> -
>
> Key: QPID-8204
> URL: https://issues.apache.org/jira/browse/QPID-8204
> Project: Qpid
>  Issue Type: Improvement
>  Components: Broker-J
>Reporter: Alex Rudyy
>Assignee: Alex Rudyy
>Priority: Major
> Fix For: qpid-java-broker-7.1.0, qpid-java-broker-7.0.5
>
> Attachments: 
> 0001-QPID-8204-Broker-J-Rename-new-statistics-into-inboun.patch
>
>
> Add statistics to report the maximum size of incoming messages on Broker and 
> Virtual Hosts. Add a switch to enable/disable collection of statistics for 
> message maximum size . By default, collection of maximum size statistics 
> should be disabled



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (QPIDJMS-386) failover provider prevents handling of remote resource closure from completing correctly

2018-06-07 Thread Daniel Maier (JIRA)


[ 
https://issues.apache.org/jira/browse/QPIDJMS-386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504354#comment-16504354
 ] 

Daniel Maier commented on QPIDJMS-386:
--

I tested the last fix with the scenario described in QPIDJMS-376. Failover 
works smooth for me now. Tanks a lot. Great work!

> failover provider prevents handling of remote resource closure from 
> completing correctly
> 
>
> Key: QPIDJMS-386
> URL: https://issues.apache.org/jira/browse/QPIDJMS-386
> Project: Qpid JMS
>  Issue Type: Bug
>  Components: qpid-jms-client
>Affects Versions: 0.32.0
> Environment:  
>Reporter: Robbie Gemmell
>Assignee: Timothy Bish
>Priority: Major
> Fix For: 0.33.0
>
>
> QPIDJMS-376 added functionality to fire the connection ExceptionListener if a 
> MessageConsumer with a MessageListener is remotely closed, since the lack of 
> future message deliveries would mean the application has no further 
> interaction with the consumer to observe the issue. It has since been 
> identified that this doesn't occur when using the built in failover provider, 
> which is preventing the process of handling the remote consumer resource 
> closure from completing fully.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org