Proton Version Detection / Changing Function Calls

2013-07-18 Thread Frank Quinn
Hi Folks,

Please bear with me as I have a few questions to pose to the mailing list but 
I'm going to send them in separate emails to avoid cross-talk on different 
topics...

I noticed recently that in your SVN copy of proton, pn_messenger_send has been 
modified to support providing another parameter. When I tried to support both 
implementations, I could find no reliable way to do so  because:

1. Version numbers are currently the same for both 0.4 and the latest off SVN
2. The version as generated to produce pn_config.h (even if it was different) 
does not seem to be distributed during the build so when building against 
compiled libraries, there doesn't seem to be any way to detect the version of 
the qpid proton libraries

Maybe I'm missing something and the version information is to be found 
elsewhere? If not, could you consider adding this?

Cheers,
Frank



Please consider the environment before printing this e-mail.

This e-mail may contain confidential and/or privileged information. If you are 
not the intended recipient or have received this e-mail in error, please advise 
the sender immediately by reply e-mail and delete this message and any 
attachments without retaining a copy.

Any unauthorised copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden.
--

Please consider the environment before printing this email.

Visit our website at http://www.nyse.com http://www.nyse.com
*
Note: The information contained in this message and any attachment to it is 
privileged, confidential and protected from disclosure. If the reader of this 
message is not the intended recipient, or an employee or agent responsible for 
delivering this message to the intended recipient, you are hereby notified that 
any dissemination, distribution or copying of this communication is strictly 
prohibited. If you have received this communication in error, please notify the 
sender immediately by replying to the message, and please delete it from your 
system. Thank you. NYSE Euronext.


[jira] [Commented] (PROTON-351) Proton-J engine violates link credit window

2013-07-18 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/PROTON-351?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712189#comment-13712189
 ] 

ASF subversion and git services commented on PROTON-351:


Commit 1504407 from r...@apache.org in branch 'proton/trunk'
[ https://svn.apache.org/r1504407 ]

Bounded messenger's credit allocation when an unlimited value is passed to 
recv; added tests for pushback at both messenger and engine level; fixed 
proton-j engine to respect credit window; fixed java messenger to properly 
report if work has been done. This addresses PROTON-350 and PROTON-351.

 Proton-J engine violates link credit window
 ---

 Key: PROTON-351
 URL: https://issues.apache.org/jira/browse/PROTON-351
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Affects Versions: 0.4
Reporter: Rafael H. Schloming
Assignee: Rafael H. Schloming
 Fix For: 0.5




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (PROTON-350) Messenger - pn_messenger_recv( -1 ) does not flow control senders

2013-07-18 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/PROTON-350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712188#comment-13712188
 ] 

ASF subversion and git services commented on PROTON-350:


Commit 1504407 from r...@apache.org in branch 'proton/trunk'
[ https://svn.apache.org/r1504407 ]

Bounded messenger's credit allocation when an unlimited value is passed to 
recv; added tests for pushback at both messenger and engine level; fixed 
proton-j engine to respect credit window; fixed java messenger to properly 
report if work has been done. This addresses PROTON-350 and PROTON-351.

 Messenger - pn_messenger_recv( -1 ) does not flow control senders
 -

 Key: PROTON-350
 URL: https://issues.apache.org/jira/browse/PROTON-350
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-c
Affects Versions: 0.4
Reporter: Ken Giusti
 Fix For: 0.5

 Attachments: test-recv.py


 When using pn_messenger_recv( -1 ), remote senders are not back-pressured, 
 and the number of messages received can grow without bounds.  It appears that 
 messenger is unconditionally granting credit as messages arrive.
 Messenger should instead only replenish credit when received messages are 
 actually consumed by the application (maybe after they are settled?).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Resolved] (PROTON-351) Proton-J engine violates link credit window

2013-07-18 Thread Rafael H. Schloming (JIRA)

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

Rafael H. Schloming resolved PROTON-351.


Resolution: Fixed

 Proton-J engine violates link credit window
 ---

 Key: PROTON-351
 URL: https://issues.apache.org/jira/browse/PROTON-351
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Affects Versions: 0.4
Reporter: Rafael H. Schloming
Assignee: Rafael H. Schloming
 Fix For: 0.5




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (PROTON-350) Messenger - pn_messenger_recv( -1 ) does not flow control senders

2013-07-18 Thread Rafael H. Schloming (JIRA)

[ 
https://issues.apache.org/jira/browse/PROTON-350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712193#comment-13712193
 ] 

Rafael H. Schloming commented on PROTON-350:


I've submitted a fix for this and added a test. Can you verify that this also 
fixes your reproducer?

 Messenger - pn_messenger_recv( -1 ) does not flow control senders
 -

 Key: PROTON-350
 URL: https://issues.apache.org/jira/browse/PROTON-350
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-c
Affects Versions: 0.4
Reporter: Ken Giusti
 Fix For: 0.5

 Attachments: test-recv.py


 When using pn_messenger_recv( -1 ), remote senders are not back-pressured, 
 and the number of messages received can grow without bounds.  It appears that 
 messenger is unconditionally granting credit as messages arrive.
 Messenger should instead only replenish credit when received messages are 
 actually consumed by the application (maybe after they are settled?).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Resolved] (PROTON-295) recv(-1) + incoming_window == bad

2013-07-18 Thread Rafael H. Schloming (JIRA)

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

Rafael H. Schloming resolved PROTON-295.


Resolution: Fixed

 recv(-1) + incoming_window == bad
 -

 Key: PROTON-295
 URL: https://issues.apache.org/jira/browse/PROTON-295
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-c
Affects Versions: 0.5
Reporter: michael goulish
Assignee: Rafael H. Schloming
 Fix For: 0.5


 Use of recv(-1) could receive enough messages that some would exceed the 
 incoming window size and be automatically accepted -- with app logic never 
 getting a say in the matter.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


Usage of pn_message_properties and pn_message_annotations

2013-07-18 Thread Frank Quinn
Hi Folks,

I am looking for ways to inject meta data into a pn_message_t in a message body 
agnostic way as I send it across a messenger interface. I have found 
pn_message_properties and pn_message_annotations and it is unclear to me what 
their purpose is.

Key questions are:

1. Will data inserted in here still survive for all proposed data types (e.g. 
will this still be a pn_data_t accessible even if the message body is json?)
2. Should I as an application developer be writing to these structures at all?
3. If the answer to #2 is yes, can properties be used to pass parameters to the 
broker such as {create:always, delete:always} which I have seen in other 
implementations but I'm not sure how to construct an equivalent instruction in C
4. Again if the answer to #2 is yes, are there any particular data types which 
should be used so it doesn't interfere with the underlying workings of 
pn_messenger or confuse the broker?

Cheers,
Frank



Please consider the environment before printing this e-mail.

This e-mail may contain confidential and/or privileged information. If you are 
not the intended recipient or have received this e-mail in error, please advise 
the sender immediately by reply e-mail and delete this message and any 
attachments without retaining a copy.

Any unauthorised copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden.
--

Please consider the environment before printing this email.

Visit our website at http://www.nyse.com http://www.nyse.com
*
Note: The information contained in this message and any attachment to it is 
privileged, confidential and protected from disclosure. If the reader of this 
message is not the intended recipient, or an employee or agent responsible for 
delivering this message to the intended recipient, you are hereby notified that 
any dissemination, distribution or copying of this communication is strictly 
prohibited. If you have received this communication in error, please notify the 
sender immediately by replying to the message, and please delete it from your 
system. Thank you. NYSE Euronext.


[jira] [Resolved] (PROTON-358) [Proton-j] CategoryLoggerDiscoveryTest fails when run on Java 7 due to tests being run in a different order

2013-07-18 Thread Rob Godfrey (JIRA)

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

Rob Godfrey resolved PROTON-358.


Resolution: Fixed

 [Proton-j] CategoryLoggerDiscoveryTest fails when run on Java 7 due to tests 
 being run in a different order
 ---

 Key: PROTON-358
 URL: https://issues.apache.org/jira/browse/PROTON-358
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Reporter: Rob Godfrey
Assignee: Rob Godfrey

 Some tests in CategoryLoggerDiscoveryTest change the default logger, but do 
 not restore the default at the end of the test.  If these tests are run 
 before the test which checks the default logger is StdOut that that test 
 fails.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (PROTON-358) [Proton-j] CategoryLoggerDiscoveryTest fails when run on Java 7 due to tests being run in a different order

2013-07-18 Thread Rob Godfrey (JIRA)
Rob Godfrey created PROTON-358:
--

 Summary: [Proton-j] CategoryLoggerDiscoveryTest fails when run on 
Java 7 due to tests being run in a different order
 Key: PROTON-358
 URL: https://issues.apache.org/jira/browse/PROTON-358
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Reporter: Rob Godfrey
Assignee: Rob Godfrey


Some tests in CategoryLoggerDiscoveryTest change the default logger, but do not 
restore the default at the end of the test.  If these tests are run before the 
test which checks the default logger is StdOut that that test fails.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (PROTON-359) [Proton-j] Create a simple entry point for creating all Proton entities

2013-07-18 Thread Rob Godfrey (JIRA)
Rob Godfrey created PROTON-359:
--

 Summary: [Proton-j] Create a simple entry point for creating all 
Proton entities
 Key: PROTON-359
 URL: https://issues.apache.org/jira/browse/PROTON-359
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Reporter: Rob Godfrey
Assignee: Rob Godfrey


Rather than requiring users to create a ProtonFactoryLoader, get a factory and 
then create an object, instead just do Proton.connection(), Proton.message(), 
etc.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (PROTON-359) [Proton-j] Create a simple entry point for creating all Proton entities

2013-07-18 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/PROTON-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712759#comment-13712759
 ] 

ASF subversion and git services commented on PROTON-359:


Commit 1504633 from [~godfrer] in branch 'proton/trunk'
[ https://svn.apache.org/r1504633 ]

PROTON-359 : [Proton-j] Create a simple entry point for creating all Proton 
entities

 [Proton-j] Create a simple entry point for creating all Proton entities
 ---

 Key: PROTON-359
 URL: https://issues.apache.org/jira/browse/PROTON-359
 Project: Qpid Proton
  Issue Type: Bug
  Components: proton-j
Reporter: Rob Godfrey
Assignee: Rob Godfrey

 Rather than requiring users to create a ProtonFactoryLoader, get a factory 
 and then create an object, instead just do Proton.connection(), 
 Proton.message(), etc.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira