Author: kpvdr Date: Tue Sep 22 18:15:57 2009 New Revision: 817748 URL: http://svn.apache.org/viewvc?rev=817748&view=rev Log: Minor update to allow store access to the message isPersistent() method. This is required for correctly setting the transient flag.
Modified: qpid/trunk/qpid/cpp/src/qpid/broker/Message.cpp qpid/trunk/qpid/cpp/src/qpid/broker/Message.h qpid/trunk/qpid/cpp/src/qpid/broker/PersistableMessage.h qpid/trunk/qpid/cpp/src/tests/QueueTest.cpp Modified: qpid/trunk/qpid/cpp/src/qpid/broker/Message.cpp URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/broker/Message.cpp?rev=817748&r1=817747&r2=817748&view=diff ============================================================================== --- qpid/trunk/qpid/cpp/src/qpid/broker/Message.cpp (original) +++ qpid/trunk/qpid/cpp/src/qpid/broker/Message.cpp Tue Sep 22 18:15:57 2009 @@ -98,7 +98,7 @@ return getAdapter().getApplicationHeaders(frames); } -bool Message::isPersistent() +bool Message::isPersistent() const { return (getAdapter().isPersistent(frames) || forcePersistentPolicy); } Modified: qpid/trunk/qpid/cpp/src/qpid/broker/Message.h URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/broker/Message.h?rev=817748&r1=817747&r2=817748&view=diff ============================================================================== --- qpid/trunk/qpid/cpp/src/qpid/broker/Message.h (original) +++ qpid/trunk/qpid/cpp/src/qpid/broker/Message.h Tue Sep 22 18:15:57 2009 @@ -74,7 +74,7 @@ bool isImmediate() const; QPID_BROKER_EXTERN const framing::FieldTable* getApplicationHeaders() const; framing::FieldTable& getOrInsertHeaders(); - QPID_BROKER_EXTERN bool isPersistent(); + QPID_BROKER_EXTERN bool isPersistent() const; bool requiresAccept(); QPID_BROKER_EXTERN void setTimestamp(const boost::intrusive_ptr<ExpiryPolicy>& e); Modified: qpid/trunk/qpid/cpp/src/qpid/broker/PersistableMessage.h URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/broker/PersistableMessage.h?rev=817748&r1=817747&r2=817748&view=diff ============================================================================== --- qpid/trunk/qpid/cpp/src/qpid/broker/PersistableMessage.h (original) +++ qpid/trunk/qpid/cpp/src/qpid/broker/PersistableMessage.h Tue Sep 22 18:15:57 2009 @@ -110,7 +110,9 @@ void requestContentRelease(); void blockContentRelease(); bool checkContentReleasable(); - + + virtual QPID_BROKER_EXTERN bool isPersistent() const = 0; + QPID_BROKER_EXTERN bool isEnqueueComplete(); QPID_BROKER_EXTERN void enqueueComplete(); Modified: qpid/trunk/qpid/cpp/src/tests/QueueTest.cpp URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/tests/QueueTest.cpp?rev=817748&r1=817747&r2=817748&view=diff ============================================================================== --- qpid/trunk/qpid/cpp/src/tests/QueueTest.cpp (original) +++ qpid/trunk/qpid/cpp/src/tests/QueueTest.cpp Tue Sep 22 18:15:57 2009 @@ -817,7 +817,7 @@ BOOST_CHECK_EQUAL(msg02->isContentReleased(), false); BOOST_CHECK_EQUAL(1u, tq1->getMessageCount()); - intrusive_ptr<Message> msg03 = mkMsg(testStore, std::string(5, 'X'), true); // transient w/ content + intrusive_ptr<Message> msg03 = mkMsg(testStore, std::string(5, 'X'), true); // durable w/ content DeliverableMessage dmsg03(msg03); BOOST_CHECK_THROW(sbtFanout1.route(dmsg03, "", 0), ResourceLimitExceededException); msg03->tryReleaseContent(); --------------------------------------------------------------------- Apache Qpid - AMQP Messaging Implementation Project: http://qpid.apache.org Use/Interact: mailto:commits-subscr...@qpid.apache.org