[jira] Updated: (QPID-2370) TTR-Qpid-07-NA failed due to channel error 504 during tear down

2010-02-01 Thread Martin Ritchie (JIRA)

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

Martin Ritchie updated QPID-2370:
-

Attachment: QPID-2370.CheckClosedBeforeFlow.patch

Added patch that causes a session.closed/closing check to be performed before 
attempting to send the ChannelFlow method.

This will not remove the race condition however, it will greatly reduce the 
chances of a ChannelFlow method occuring AFTER a ChannelClose has been sent.

 TTR-Qpid-07-NA failed due to channel error 504 during tear down
 ---

 Key: QPID-2370
 URL: https://issues.apache.org/jira/browse/QPID-2370
 Project: Qpid
  Issue Type: Bug
  Components: Java Performance Tests
Affects Versions: 0.6
Reporter: Martin Ritchie
Assignee: Martin Ritchie
 Attachments: QPID-2370.CheckClosedBeforeFlow.patch


 During a full 'RunCore.sh' performance test run TTR-Qpid-07-NA hung holding 
 the test run up. This appeared to be as a result of a Channel Error that was 
 thrown during. 
 Thread-4 2010-01-26 03:29:48,722 ERROR [apache.qpid.client.AMQConnection] 
 error:
 org.apache.qpid.AMQConnectionClosedException: Error: channel is closed [error 
 code 504: channel error] [error code 504: channel error]
 This may have occured during test tear down as the following was logged by 
 the same thread just after the stack trace.
 Thread-4 2010-01-26 03:29:48,725 WARN [apache.qpid.ping.PingTestPerf] There 
 was an exception during per thread tear down.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Updated: (QPID-2370) TTR-Qpid-07-NA failed due to channel error 504 during tear down

2010-02-01 Thread Martin Ritchie (JIRA)

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

Martin Ritchie updated QPID-2370:
-

Attachment: (was: QPID-2370.CheckClosedBeforeFlow.patch)

 TTR-Qpid-07-NA failed due to channel error 504 during tear down
 ---

 Key: QPID-2370
 URL: https://issues.apache.org/jira/browse/QPID-2370
 Project: Qpid
  Issue Type: Bug
  Components: Java Performance Tests
Affects Versions: 0.6
Reporter: Martin Ritchie
Assignee: Martin Ritchie
 Attachments: QPID-2370.CheckClosedBeforeFlow.patch


 During a full 'RunCore.sh' performance test run TTR-Qpid-07-NA hung holding 
 the test run up. This appeared to be as a result of a Channel Error that was 
 thrown during. 
 Thread-4 2010-01-26 03:29:48,722 ERROR [apache.qpid.client.AMQConnection] 
 error:
 org.apache.qpid.AMQConnectionClosedException: Error: channel is closed [error 
 code 504: channel error] [error code 504: channel error]
 This may have occured during test tear down as the following was logged by 
 the same thread just after the stack trace.
 Thread-4 2010-01-26 03:29:48,725 WARN [apache.qpid.ping.PingTestPerf] There 
 was an exception during per thread tear down.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Updated: (QPID-2370) TTR-Qpid-07-NA failed due to channel error 504 during tear down

2010-02-01 Thread Martin Ritchie (JIRA)

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

Martin Ritchie updated QPID-2370:
-

Attachment: QPID-2370.CheckClosedBeforeFlow.patch

I did mean to grant permission

 TTR-Qpid-07-NA failed due to channel error 504 during tear down
 ---

 Key: QPID-2370
 URL: https://issues.apache.org/jira/browse/QPID-2370
 Project: Qpid
  Issue Type: Bug
  Components: Java Performance Tests
Affects Versions: 0.6
Reporter: Martin Ritchie
Assignee: Martin Ritchie
 Attachments: QPID-2370.CheckClosedBeforeFlow.patch


 During a full 'RunCore.sh' performance test run TTR-Qpid-07-NA hung holding 
 the test run up. This appeared to be as a result of a Channel Error that was 
 thrown during. 
 Thread-4 2010-01-26 03:29:48,722 ERROR [apache.qpid.client.AMQConnection] 
 error:
 org.apache.qpid.AMQConnectionClosedException: Error: channel is closed [error 
 code 504: channel error] [error code 504: channel error]
 This may have occured during test tear down as the following was logged by 
 the same thread just after the stack trace.
 Thread-4 2010-01-26 03:29:48,725 WARN [apache.qpid.ping.PingTestPerf] There 
 was an exception during per thread tear down.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2371) Missing includes for MSVC 2005 compiler

2010-02-01 Thread Marcel Roelofs (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828105#action_12828105
 ] 

Marcel Roelofs commented on QPID-2371:
--

Steve, everything compiles out of the box with the patch to the cmakefile 
below. The extra define specifies that the compiler targets Windows XP and 
Windows Server 2003; it's necessary to pull some extra declarations into the 
Windows header files that are needed to compile qpid. Apparently this (or 
better) is already the default for VC9 but not for VC8. If the default for VC9 
is higher than I guess taking the VC9 default value won't hurt either.

--- cpp/src/CMakeLists.txt  (revision 904026)
+++ cpp/src/CMakeLists.txt  (working copy)
@@ -404,6 +404,7 @@
   /D NOMINMAX
   /D WIN32_LEAN_AND_MEAN
   /D _SCL_SECURE_NO_WARNINGS
+  /D _WIN32_WINNT=0x0501
   /wd4244
   /wd4800
   /wd4355


 Missing includes for MSVC 2005 compiler
 ---

 Key: QPID-2371
 URL: https://issues.apache.org/jira/browse/QPID-2371
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Broker
Affects Versions: 0.6
 Environment: MSVC 2005
Reporter: Marcel Roelofs
Assignee: Steve Huston
Priority: Minor
 Fix For: 0.7

 Attachments: msvc2005.patch


 When compiling the 0.6 release candidate using MSVC 2005 I found some missing 
 header file includes that prevented qpid to compile out of the box. I'll add 
 a patch later.
 Also I had to add /D _WIN32_WINNT=0x0501 to the compiler flags during cmake 
 makefile generation to prevent compile errors in various components, but I've 
 no idea how to add this to the cmake configuration. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Updated: (QPID-2370) TTR-Qpid-07-NA failed due to channel error 504 during tear down

2010-02-01 Thread Martin Ritchie (JIRA)

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

Martin Ritchie updated QPID-2370:
-

Attachment: QPID-2370.CheckClosedBeforeFlowWithStateCheck.patch

Updated the patch to check the state of the suspension to before creating a 
thread. This will reduce thread creations.

 TTR-Qpid-07-NA failed due to channel error 504 during tear down
 ---

 Key: QPID-2370
 URL: https://issues.apache.org/jira/browse/QPID-2370
 Project: Qpid
  Issue Type: Bug
  Components: Java Performance Tests
Affects Versions: 0.6
Reporter: Martin Ritchie
Assignee: Martin Ritchie
 Attachments: QPID-2370-Improve-BrokerDebug.patch, 
 QPID-2370.CheckClosedBeforeFlow.patch, 
 QPID-2370.CheckClosedBeforeFlowWithStateCheck.patch


 During a full 'RunCore.sh' performance test run TTR-Qpid-07-NA hung holding 
 the test run up. This appeared to be as a result of a Channel Error that was 
 thrown during. 
 Thread-4 2010-01-26 03:29:48,722 ERROR [apache.qpid.client.AMQConnection] 
 error:
 org.apache.qpid.AMQConnectionClosedException: Error: channel is closed [error 
 code 504: channel error] [error code 504: channel error]
 This may have occured during test tear down as the following was logged by 
 the same thread just after the stack trace.
 Thread-4 2010-01-26 03:29:48,725 WARN [apache.qpid.ping.PingTestPerf] There 
 was an exception during per thread tear down.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2364) Management updates in timer create inconsistencies in a cluster.

2010-02-01 Thread Alan Conway (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2364?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828113#action_12828113
 ] 

Alan Conway commented on QPID-2364:
---

Refactoring done in  r904656


 Management updates in timer create inconsistencies in a cluster.
 

 Key: QPID-2364
 URL: https://issues.apache.org/jira/browse/QPID-2364
 Project: Qpid
  Issue Type: Bug
  Components: C++ Broker
Affects Versions: 0.6
Reporter: Alan Conway
Assignee: Alan Conway
 Attachments: periodic-timer.patch


 Management updates are triggered by a timer. They are not predictable for the
 cluster and so can cause cluster shut-downs and inconsistent message delivery.
 We have a hack in place that suppresses exceptions when the session receives
 completions for transfers not yet sent (which is the usual manifestation of 
 the
 unpredictability). I.e. we have in essence disabled consistency checking for
 management sessions. This solved immediate problems but would quickly stop
 working if sessions/connections could be used for management and other things
 (as will be more likely with QMFv2 where using management becomes quite
 straightforward).
 In a cluster, management updates need to be synchronized by executing them in 
 the cluster dispatch thread rather than a timer thread.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Resolved: (QPID-2348) [C++] The HeadersExchange does not support federation

2010-02-01 Thread Ted Ross (JIRA)

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

Ted Ross resolved QPID-2348.


Resolution: Fixed

 [C++] The HeadersExchange does not support federation
 -

 Key: QPID-2348
 URL: https://issues.apache.org/jira/browse/QPID-2348
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Broker
Affects Versions: 0.6
 Environment: all environments
Reporter: Sam Joyce
Assignee: Ted Ross
Priority: Minor
 Fix For: 0.7

 Attachments: fed_headers_exchange-2010-01-27.patch

   Original Estimate: 0h
  Remaining Estimate: 0h

 (from Ted Ross) 
 Currently broker federation only deals with binding keys.  Both dynamic and
 static federation do not support exchanges which use argument-map data in 
 their
 bindings (i.e. XML, Headers, unknown future extension exchange).
 Support for static federation with arguments needs to be added.
 Support for dynamic federation that propagates bindings with arguments needs 
 to
 be added.
 (from Sam Joyce)
 This bug actually breaks down into 4 separate tasks:
 1) Allow the exchange to accept dynamic bindings
 2) Allow the exchange to be used by the qpid-route tool to create dynamic
 routes
 3) propagate bind and unbind requests to federated peers
 4) support reOrigin requests in the event the things get into a bit of a flap
 (flap being a technical term :) )

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Created: (QPID-2380) define and implement reliability options for senders and receivers

2010-02-01 Thread Gordon Sim (JIRA)
define and implement reliability options for senders and receivers
--

 Key: QPID-2380
 URL: https://issues.apache.org/jira/browse/QPID-2380
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Client, Python Client
Affects Versions: 0.6
Reporter: Gordon Sim
Assignee: Rafael H. Schloming


c++ client currently only recognises 'reliability' option for receivers 
(unreliable and at-most-once are handled by no-acks and by auto-deleting temp 
subscription queues on failover)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2251) Provide portable signal mechanism for the C++ QMF agent API

2010-02-01 Thread Pete MacKinnon (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828183#action_12828183
 ] 

Pete MacKinnon commented on QPID-2251:
--

Should/could we wrap the type returned from getSignalFd()? Private posix impl 
on the wrapper returns int, private windows impl returns SOCKET.

\Pete

 Provide portable signal mechanism for the C++ QMF agent API
 ---

 Key: QPID-2251
 URL: https://issues.apache.org/jira/browse/QPID-2251
 Project: Qpid
  Issue Type: Improvement
  Components: Qpid Managment Framework
 Environment: Windows XP SP3, VC++ 9.0
 RHEL, Fedora
Reporter: Pete MacKinnon
Assignee: Ted Ross
 Fix For: 0.7


 Need a portable signaling mechanism in the QMF agent API for Windows and 
 Posix platforms that encapsulates or hides socket and FD implementations

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Created: (QPID-2381) finalise api around synchronising receiver and sender creation

2010-02-01 Thread Gordon Sim (JIRA)
finalise api around synchronising receiver and sender creation
--

 Key: QPID-2381
 URL: https://issues.apache.org/jira/browse/QPID-2381
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Client, Python Client
Affects Versions: 0.6
Reporter: Gordon Sim
Assignee: Rafael H. Schloming


The sender() and receiver() calls in the python are not synchronous - on return 
there is no guarantee that any action has yet been take on the broker or even 
that the address has been confirmed as for a valid node. Further there is then 
no obvious way (other than trying to send or fetch) to actually wait until any 
action has succeeded.

On the c++ client the createReceiver() and createSender() calls will only 
return once the necessary server actions have completed successfully.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Created: (QPID-2382) implement utility for receiving member updates from failover exchange

2010-02-01 Thread Gordon Sim (JIRA)
implement utility for receiving member updates from failover exchange
-

 Key: QPID-2382
 URL: https://issues.apache.org/jira/browse/QPID-2382
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Client, Python Client
Reporter: Gordon Sim
Assignee: Rafael H. Schloming




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2380) define and implement reliability options for senders and receivers

2010-02-01 Thread Gordon Sim (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828192#action_12828192
 ] 

Gordon Sim commented on QPID-2380:
--

From doxygen for c++ client at present:

   reliability - indicates the level of reliability that the receiver expects. 
Can be one of unreliable, at-most-once, at-least-once or exactly-once (the 
latter is not yet correctly supported)

unreliable and at-most-once are currently treated as the same and for a 
receiver they will mean that accept-mode=none is used (for 0-10) and in the 
case of receiving from an exchange an auto-deleted subscription queue is used 
meaning that messages can be missed during failover or otherwise when not 
connected. For senders these mean that an outgoing message is no longer 
considered pending when it has been written to the wire (regardless of whether 
the broker has yet received it) [Note: in the current c++ client impl this is 
not possible and a temporary workaround will be put in place]

at-least-once uses accept-mode=explicit (0-10) for receivers and a queue that 
is not deleted when the session is lost for topic receivers; for senders it 
results in a replay buffer being maintained and messages replayed when a 
disconnected connection is reconnected (e.g. on failover).



 define and implement reliability options for senders and receivers
 --

 Key: QPID-2380
 URL: https://issues.apache.org/jira/browse/QPID-2380
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Client, Python Client
Affects Versions: 0.6
Reporter: Gordon Sim
Assignee: Rafael H. Schloming

 c++ client currently only recognises 'reliability' option for receivers 
 (unreliable and at-most-once are handled by no-acks and by auto-deleting temp 
 subscription queues on failover)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



Re: svn commit: r904566 - /qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java

2010-02-01 Thread Martin Ritchie
Rajith,

Can we have some tests to validate the these new SASL connections?

Cheers

Martin

On 29 January 2010 17:30,  raj...@apache.org wrote:
 Author: rajith
 Date: Fri Jan 29 17:30:39 2010
 New Revision: 904566

 URL: http://svn.apache.org/viewvc?rev=904566view=rev
 Log:
 Corrected a mistake I made in rev904375

 Modified:
    
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java

 Modified: 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
 URL: 
 http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java?rev=904566r1=904565r2=904566view=diff
 ==
 --- 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
  (original)
 +++ 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
  Fri Jan 29 17:30:39 2010
 @@ -72,7 +72,7 @@
     {
         Connection conn = connection();

 -        if (conn.getConnectionSettings() != null 
 +        if (conn.getConnectionSettings() != null 
             conn.getConnectionSettings().isUseSASLEncryption())
         {
             sender = new SASLSender(sender);
 @@ -87,7 +87,7 @@

     public ReceiverByteBuffer receiver(Connection conn)
     {
 -        if (conn.getConnectionSettings() != null 
 +        if (conn.getConnectionSettings() != null 
             conn.getConnectionSettings().isUseSASLEncryption())
         {
             SASLReceiver receiver = new SASLReceiver(new InputHandler(new 
 Assembler(conn)));



 -
 Apache Qpid - AMQP Messaging Implementation
 Project:      http://qpid.apache.org
 Use/Interact: mailto:commits-subscr...@qpid.apache.org





-- 
Martin Ritchie

-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2367) Early Initialization of File Descriptors Conflicts With Daemon Best Practices

2010-02-01 Thread Jason Schlauch (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828278#action_12828278
 ] 

Jason Schlauch commented on QPID-2367:
--

Yes, you don't need to handle anything other than stdout, stdin, and stderr.  
However, for whatever reason, boilerplate daemon code often suggests (and goes 
on to implement) closing all file descriptors.  So perhaps best practice 
wasn't quite accurate, and common practice or it was in the code you 
inherited might be better.

I handled this by simply changing the daemon code to close only stdout, stdin, 
and stderr.  Finding the error, however, took up a fair chunk of time.  Some 
defensive coding in the QPID client might save others the same trouble.

For example, could you simply check if the fd was open before attempting to use 
it (and reopen it if it's not)?

 Early Initialization of File Descriptors Conflicts With Daemon Best Practices
 -

 Key: QPID-2367
 URL: https://issues.apache.org/jira/browse/QPID-2367
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Client
Affects Versions: 0.5
 Environment: Linux (possibly all UNIX), c++, g++
Reporter: Jason Schlauch

 At least one file descriptor (in qpid/sys/epoll/EpollPoller.*) in the c++ 
 client is global and declared as static.  In programs linked against the c++ 
 qpid libs g++ generates code for allocation and, more importantly, 
 initialization of these descriptors that occurs before main().  You can 
 confirm this with gdb by breakpointing both the initialization and main() 
 (the initialization break is hit first).
 On the other hand, the canonical recipe for creating a UNIX daemon calls for 
 the closing of all open file descriptors after fork()ing (where the fork() 
 certainly occurs after main()).  While not an absolute requirement, closing 
 all open file descriptors is considered a best practice.  A loop to close all 
 descriptors is also common in boilerplate daemon creation code and has 
 undoubtedly been cut-and-pasted into numerous daemons.
 The net effect is that the typical daemon will close the file descriptor 
 opened before main() in the c++ client library.  In the case of the epoll 
 code this manifests as an inability to connect to the broker.
 A fix for this would be to defer the initialization of the file descriptor 
 (perhaps via the Singleton pattern or a move of the variables into a class 
 member).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2368) Exception Thrown at qpid/sys/epoll/EpollPoller.cpp:254 Leaves Orphan File Descriptor

2010-02-01 Thread Jason Schlauch (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828286#action_12828286
 ] 

Jason Schlauch commented on QPID-2368:
--

How about:

try {
QPID_POSIX_CHECK(::epoll_ctl(epollFd, EPOLL_CTL_ADD, alwaysReadableFd, epe)); 
} catch (...) {
   close(epollFd);
   throw;
}

 Exception Thrown at qpid/sys/epoll/EpollPoller.cpp:254 Leaves Orphan File 
 Descriptor
 

 Key: QPID-2368
 URL: https://issues.apache.org/jira/browse/QPID-2368
 Project: Qpid
  Issue Type: Bug
  Components: C++ Client
Affects Versions: 0.5
 Environment: c++ client
Reporter: Jason Schlauch
Priority: Minor

 While researching JIRA QPID-2367 I noticed a pileup of file descriptors in 
 /proc/PID/fd.  I traced the creation of these descriptors back to this chunk 
 of code in qpid/sys/epoll/EpollPoller.cpp:
 244 PollerPrivate() :
 245 epollFd(::epoll_create(DefaultFds)),
 246 isShutdown(false) {
 247 QPID_POSIX_CHECK(epollFd);
 248 ::sigemptyset(sigMask);
 249 // Add always readable fd into our set (but not listening to it 
 yet)
 250 ::epoll_event epe;
 251 epe.events = 0;
 252 epe.data.u64 = 0;
 253 QPID_POSIX_CHECK(::epoll_ctl(epollFd, EPOLL_CTL_ADD, 
 alwaysReadableFd, epe));
 254 }
 The problem is with the second QPID_POSIX_CHECK -- a macro that throws an 
 exception.  If an exception is thrown then the file descriptor allocated by 
 epollFd(::epoll_create(DefaultFds)) is left dangling.  A ::close(epollFd) 
 would be needed in the catch() block to free it.  
 There are a number of functions with a similar design in EpollPoller.cpp that 
 might be similarly affected.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



Re: svn commit: r904566 - /qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java

2010-02-01 Thread Rajith Attapattu
As mentioned in the original commit message, the SASL encryption is
not fully functional.
I committed the code in the hopes of working with the developers of
the c++ broker to diagnose the issue.
There is no harm due to it being inactive unless explicitly enabled.

As for tests, as soon as the issue is resolved, I will be adding a new
test profile similar to the SSL profile.

Rajith

On Mon, Feb 1, 2010 at 1:47 PM, Martin Ritchie ritch...@apache.org wrote:
 Rajith,

 Can we have some tests to validate the these new SASL connections?

 Cheers

 Martin

 On 29 January 2010 17:30,  raj...@apache.org wrote:
 Author: rajith
 Date: Fri Jan 29 17:30:39 2010
 New Revision: 904566

 URL: http://svn.apache.org/viewvc?rev=904566view=rev
 Log:
 Corrected a mistake I made in rev904375

 Modified:
    
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java

 Modified: 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
 URL: 
 http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java?rev=904566r1=904565r2=904566view=diff
 ==
 --- 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
  (original)
 +++ 
 qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/network/ConnectionBinding.java
  Fri Jan 29 17:30:39 2010
 @@ -72,7 +72,7 @@
     {
         Connection conn = connection();

 -        if (conn.getConnectionSettings() != null 
 +        if (conn.getConnectionSettings() != null 
             conn.getConnectionSettings().isUseSASLEncryption())
         {
             sender = new SASLSender(sender);
 @@ -87,7 +87,7 @@

     public ReceiverByteBuffer receiver(Connection conn)
     {
 -        if (conn.getConnectionSettings() != null 
 +        if (conn.getConnectionSettings() != null 
             conn.getConnectionSettings().isUseSASLEncryption())
         {
             SASLReceiver receiver = new SASLReceiver(new InputHandler(new 
 Assembler(conn)));



 -
 Apache Qpid - AMQP Messaging Implementation
 Project:      http://qpid.apache.org
 Use/Interact: mailto:commits-subscr...@qpid.apache.org





 --
 Martin Ritchie

 -
 Apache Qpid - AMQP Messaging Implementation
 Project:      http://qpid.apache.org
 Use/Interact: mailto:dev-subscr...@qpid.apache.org





-- 
Regards,

Rajith Attapattu
Red Hat
http://rajith.2rlabs.com/

-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2371) Missing includes for MSVC 2005 compiler

2010-02-01 Thread Steve Huston (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828360#action_12828360
 ] 

Steve Huston commented on QPID-2371:


Thanks for checking into this, Marcel. I'd rather not lock this in for every 
build. Can do please check where this is needed? If it's pervasive, please try 
this change to cpp/src/CMakeLists.txt instead:

Index: CMakeLists.txt
===
--- CMakeLists.txt  (revision 905160)
+++ CMakeLists.txt  (working copy)
@@ -409,6 +409,9 @@
   /wd4800
   /wd4355
 )
+if (MSVC80)
+  add_definitions(/D _WIN32_WINNT=0x0501)
+endif (MSVC80)
   endif (MSVC)
 
   set (qpidcommon_platform_SOURCES


 Missing includes for MSVC 2005 compiler
 ---

 Key: QPID-2371
 URL: https://issues.apache.org/jira/browse/QPID-2371
 Project: Qpid
  Issue Type: Improvement
  Components: C++ Broker
Affects Versions: 0.6
 Environment: MSVC 2005
Reporter: Marcel Roelofs
Assignee: Steve Huston
Priority: Minor
 Fix For: 0.7

 Attachments: msvc2005.patch


 When compiling the 0.6 release candidate using MSVC 2005 I found some missing 
 header file includes that prevented qpid to compile out of the box. I'll add 
 a patch later.
 Also I had to add /D _WIN32_WINNT=0x0501 to the compiler flags during cmake 
 makefile generation to prevent compile errors in various components, but I've 
 no idea how to add this to the cmake configuration. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org



[jira] Commented: (QPID-2251) Provide portable signal mechanism for the C++ QMF agent API

2010-02-01 Thread Steve Huston (JIRA)

[ 
https://issues.apache.org/jira/browse/QPID-2251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12828363#action_12828363
 ] 

Steve Huston commented on QPID-2251:


I think the changes Ted added (if I understand them correctly) make this a moot 
point... Ted?


 Provide portable signal mechanism for the C++ QMF agent API
 ---

 Key: QPID-2251
 URL: https://issues.apache.org/jira/browse/QPID-2251
 Project: Qpid
  Issue Type: Improvement
  Components: Qpid Managment Framework
 Environment: Windows XP SP3, VC++ 9.0
 RHEL, Fedora
Reporter: Pete MacKinnon
Assignee: Ted Ross
 Fix For: 0.7


 Need a portable signaling mechanism in the QMF agent API for Windows and 
 Posix platforms that encapsulates or hides socket and FD implementations

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
Apache Qpid - AMQP Messaging Implementation
Project:  http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org