jhutchison edited a comment on pull request #6604: URL: https://github.com/apache/geode/pull/6604#issuecomment-859623385
> What's the reasoning behind the rename of the package from "pubsub" to "publishAndSubscribe"? Also, it seems like you only renamed the packages in the "test", "integrationTest" and "acceptanceTest" submodules, but not "main" and "distributedTest." In addition, the 4 classes in the "acceptanceTest" submodule still have their package as "pubsub" in the files, despite being in the "publishAndSubscribe" directory. Thanks for catching all that-concentration has been a little low this week :) As far as the rationale for the change goes: When publlsh and subscribe functionality was originally introduced in our codebase, it was referred to as "pubsub" and a lot of stuff was named accordingly. Now that we've added the specific command named "pubsub" it's unclear what the identifier "pubsub" refers to (the specific command, or the over all publish and subscribe functionality). The rename is an attempt to untangle this and start a new convention of referring to the overall functionality by the full name and reserve the term "pubsub" for stuff related to the specific command by that name. hit me up directly if you disagree with that idea, or want to discuss in more depth? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
