Steve Huston (JIRA) wrote:
API change to qpid::broker::MessageStore not portable
-----------------------------------------------------

                 Key: QPID-2211
                 URL: https://issues.apache.org/jira/browse/QPID-2211
             Project: Qpid
          Issue Type: Bug
          Components: C++ Broker
    Affects Versions: 0.6
            Reporter: Steve Huston
            Assignee: Alan Conway


The following pure virtual function was added to qpid::broker::MessageStore:

virtual std::string getStoreDir() const = 0;

Two problems as a result:

1. No corresponding change was made to the qpid/store/MessageStorePlugin.cpp, 
which is the portable persistence plug-in layer (or, that's it's purpose, 
anyway). This broker the Windows build.

2. Not all message stores have a store dir - this is (at this point, anyway) an 
option only for the Red Hat store.

Could this addition be explained please? Also, if it needs to stay as is, 
please fix the Windows build.




Coping Alan, I expect he just wants to write a marker for cluster auto restart for crash recovery... I expect he could just as well use data dir and create a generic file there maybe.

Alan would need to comment.

Carl.



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to