Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Message collapsing #9

Closed
martinthomson opened this issue Apr 2, 2015 · 9 comments
Closed

Message collapsing #9

martinthomson opened this issue Apr 2, 2015 · 9 comments

Comments

@martinthomson
Copy link
Contributor

@martinthomson martinthomson commented Apr 2, 2015

Need to do this once we have #3 and #6 are sorted out.

@brianraymor
Copy link

@brianraymor brianraymor commented Apr 8, 2015

If we decide to pursue - To keep related changes together, the pull request should include the changes for the conditional DELETE of the message resource by the user agent.

@martinthomson
Copy link
Contributor Author

@martinthomson martinthomson commented Apr 10, 2015

Defer to -01.

@martinthomson
Copy link
Contributor Author

@martinthomson martinthomson commented Jul 1, 2015

Defer again.

@brianraymor
Copy link

@brianraymor brianraymor commented Jul 19, 2015

Deferred multiple times. Closing. Can be reopened if there is a requirement to support this scenario.

@martinthomson
Copy link
Contributor Author

@martinthomson martinthomson commented Nov 3, 2015

Discussed in Yokohama: action on @martinthomson to generate a pull request that uses a string collapse key. That would allow the application server to operate in a stateless mode, something that the previous pull request would not allow.

@martinthomson martinthomson self-assigned this Nov 3, 2015
@martinthomson
Copy link
Contributor Author

@martinthomson martinthomson commented Nov 12, 2015

Just did a scan of the header fields that are already registered. The only one that is even a remote possibility for this purpose is SLUG. It looks like we might want to ask for another new one, current candidate: Topic.

@brianraymor
Copy link

@brianraymor brianraymor commented Nov 12, 2015

Topic sounds like a better option than reuse in this case.

@costinm
Copy link

@costinm costinm commented Jan 11, 2016

"Topic" may be a bit confusing, it is commonly used in pubsub. I don't know if pubsub uses a Topic header, but it may still confuse developers.

@RichardMaher
Copy link

@RichardMaher RichardMaher commented May 5, 2016

So this is where it all went wrong :-(

#87

_TOPICS are NOT collapse keys!_

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants