[jira] [Commented] (PROTON-522) Apache Qpid Proton on Mac/OSX - C/Objective-C

2017-11-24 Thread Roddie Kieley (JIRA)

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

Roddie Kieley commented on PROTON-522:
--

[~gemmellr], [~astitcher] the local build issue looks related to an old source 
or include file being picked up due to the work I'm doing on a Portfile for the 
MacPorts collection. Now working ok here with same configuration as the travis 
build.

> Apache Qpid Proton on Mac/OSX - C/Objective-C
> -
>
> Key: PROTON-522
> URL: https://issues.apache.org/jira/browse/PROTON-522
> Project: Qpid Proton
>  Issue Type: New Feature
>  Components: proton-c
>Reporter: Guy Dillen
>  Labels: osx
>
> I would like using Apache Qpid Proton-C from a C or Objective-C application 
> on Mac/OSX to connect as a client to Windows Azure Service Bus.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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



[jira] [Commented] (PROTON-1690) JMH Benchmarks for baseline performance of Message encoding/decoding

2017-11-24 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on PROTON-1690:


Github user gemmellr commented on the issue:

https://github.com/apache/qpid-proton-j/pull/12
  
Hi Franz

Thanks for the PR. I have some feedback for changes that would be good 
and/or necessary.

The JMH dependencies are not ALv2 compatible and so I think we should make 
it an optional module, which doesn't deploy anything. I think it will often be 
true people wont want to run those tests regularly enough to warrant including 
them with every build, so I think it also makes sense to make it optional for 
that reason anyway. Making it so unfortunately adds some issues around 
versioning during the release, as we will still want the module versions to get 
updated, so we need to adjust the process slightly to handle that. I'd also 
suggest that we move them under the tests directory.

I've had a stab at making the build/release work with this all in mind, 
going with tests/performance-jmh as the new directory. I've pasted a patch at 
https://paste.apache.org/lgCc with some of the changes. The changes for the 
root pom.xml are the diff from master, instead of those here rather than on 
top, while the tests/performance-jmh/pom.xml is the full new file I ended up 
with rather than a diff from here. With the updated poms, the module can be 
enabled as part of the overall build by adding -Pperformance-jmh, or it can 
just be built serparately using its own pom directly. Theres some trickery in 
there to ensure the release is performed in a way that updates the versions, or 
else fails.

The LICENSE file being added here can be removed, its covered by the one in 
the root dir already.


> JMH Benchmarks for baseline performance of Message encoding/decoding
> 
>
> Key: PROTON-1690
> URL: https://issues.apache.org/jira/browse/PROTON-1690
> Project: Qpid Proton
>  Issue Type: New Feature
>  Components: proton-j
>Reporter: Francesco Nigro
>Priority: Minor
>
> Using a standard and reliable tool to quantify the performances of critical 
> components would be useful to measure the effects of any changes in the 
> engine.
> Currently lo standard the facto for such measurements is JMH 
> (http://openjdk.java.net/projects/code-tools/jmh/) hence it is highly 
> desiderable to use it to avoid the JVM to perform un-wanted (ie unrealistic 
> in production uses) optimizations that could trick the results of any 
> handrolled measurements.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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



[GitHub] qpid-proton-j issue #12: PROTON-1690 JMH Benchmarks for baseline performance...

2017-11-24 Thread gemmellr
Github user gemmellr commented on the issue:

https://github.com/apache/qpid-proton-j/pull/12
  
Hi Franz

Thanks for the PR. I have some feedback for changes that would be good 
and/or necessary.

The JMH dependencies are not ALv2 compatible and so I think we should make 
it an optional module, which doesn't deploy anything. I think it will often be 
true people wont want to run those tests regularly enough to warrant including 
them with every build, so I think it also makes sense to make it optional for 
that reason anyway. Making it so unfortunately adds some issues around 
versioning during the release, as we will still want the module versions to get 
updated, so we need to adjust the process slightly to handle that. I'd also 
suggest that we move them under the tests directory.

I've had a stab at making the build/release work with this all in mind, 
going with tests/performance-jmh as the new directory. I've pasted a patch at 
https://paste.apache.org/lgCc with some of the changes. The changes for the 
root pom.xml are the diff from master, instead of those here rather than on 
top, while the tests/performance-jmh/pom.xml is the full new file I ended up 
with rather than a diff from here. With the updated poms, the module can be 
enabled as part of the overall build by adding -Pperformance-jmh, or it can 
just be built serparately using its own pom directly. Theres some trickery in 
there to ensure the release is performed in a way that updates the versions, or 
else fails.

The LICENSE file being added here can be removed, its covered by the one in 
the root dir already.


---

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



[jira] [Commented] (QPID-8044) [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote

2017-11-24 Thread ASF subversion and git services (JIRA)

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

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

Commit 38ce918dd39e9a193d37643bc1287a5da84e5cee in qpid-jms-amqp-0-x's branch 
refs/heads/6.3.x from [~k-wall]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-jms-amqp-0-x.git;h=38ce918 ]

QPID-8044: [JMS AMQP 0-x] Address minor packaging problems


> [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote
> --
>
> Key: QPID-8044
> URL: https://issues.apache.org/jira/browse/QPID-8044
> Project: Qpid
>  Issue Type: Bug
>  Components: JMS AMQP 0-x
>Affects Versions: qpid-java-client-0-x-6.3.0
>Reporter: Keith Wall
>Priority: Minor
> Fix For: qpid-java-client-0-x-6.3.1
>
>
> Robbie raised the following during the 6.3.0 vote:
> > Comments:
> > -The NOTICE file in the source release archive says Apache Qpid Java
> > and should be updated going forward.
> > -The slf4j-api details in the binary archive NOTICE can be removed
> > going forward as its covered by the details in the LICENCE file
> > already.
> > -The directories in all the archives dont match their file names (sans
> > extension), which seems like it was the intended name, e.g
> > apache-qpid-jms-0-x-6.3.0-src isn't aligned with
> > apache-qpid-jms-amqp-0-x-6.3.0-src.tar.gz



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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



[jira] [Assigned] (QPID-8044) [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote

2017-11-24 Thread Keith Wall (JIRA)

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

Keith Wall reassigned QPID-8044:


Assignee: Keith Wall

> [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote
> --
>
> Key: QPID-8044
> URL: https://issues.apache.org/jira/browse/QPID-8044
> Project: Qpid
>  Issue Type: Bug
>  Components: JMS AMQP 0-x
>Affects Versions: qpid-java-client-0-x-6.3.0
>Reporter: Keith Wall
>Assignee: Keith Wall
>Priority: Minor
> Fix For: qpid-java-client-0-x-6.3.1
>
>
> Robbie raised the following during the 6.3.0 vote:
> > Comments:
> > -The NOTICE file in the source release archive says Apache Qpid Java
> > and should be updated going forward.
> > -The slf4j-api details in the binary archive NOTICE can be removed
> > going forward as its covered by the details in the LICENCE file
> > already.
> > -The directories in all the archives dont match their file names (sans
> > extension), which seems like it was the intended name, e.g
> > apache-qpid-jms-0-x-6.3.0-src isn't aligned with
> > apache-qpid-jms-amqp-0-x-6.3.0-src.tar.gz



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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



[jira] [Commented] (QPID-8044) [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote

2017-11-24 Thread ASF subversion and git services (JIRA)

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

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

Commit ce644a322d9b2433fe26e29cb9df9fa9a3bb3da8 in qpid-jms-amqp-0-x's branch 
refs/heads/master from [~k-wall]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-jms-amqp-0-x.git;h=ce644a3 ]

QPID-8044: [JMS AMQP 0-x] Address minor packaging problems


> [JMS AMQP 0-x] Address minor packaging issues raised during 6.3.0 vote
> --
>
> Key: QPID-8044
> URL: https://issues.apache.org/jira/browse/QPID-8044
> Project: Qpid
>  Issue Type: Bug
>  Components: JMS AMQP 0-x
>Affects Versions: qpid-java-client-0-x-6.3.0
>Reporter: Keith Wall
>Priority: Minor
> Fix For: qpid-java-client-0-x-6.3.1
>
>
> Robbie raised the following during the 6.3.0 vote:
> > Comments:
> > -The NOTICE file in the source release archive says Apache Qpid Java
> > and should be updated going forward.
> > -The slf4j-api details in the binary archive NOTICE can be removed
> > going forward as its covered by the details in the LICENCE file
> > already.
> > -The directories in all the archives dont match their file names (sans
> > extension), which seems like it was the intended name, e.g
> > apache-qpid-jms-0-x-6.3.0-src isn't aligned with
> > apache-qpid-jms-amqp-0-x-6.3.0-src.tar.gz



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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