-
Notifications
You must be signed in to change notification settings - Fork 86
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
Clarify what it means for clients to support batch
#177
Comments
To clarify: are you talking about the spec, or about support tables? |
Echoing @DarthGandalf, clients may want to handle batches in their own way. However, as long as a client can request + handle batch commands then they they do support batch. If a client has no need to send batch requests then the support pages showing this may come across as penalising in comparison to others for no good reason. |
IMO supporting batch shouldn't involve supporting any particular batch type. It should just be requesting and supporting the infrastructure for receiving and handling batches. We can have a separate support table for specific batch types. |
Coolcool, in that case we're all pretty much on the same page, can just keep doing this in the future. I'll leave this issue open 'til #45 is resolved and add a note somewhere so we know this going forward. edit: @DarthGandalf just the support tables yeah |
It's pretty explicit when servers support batches. Q: Do they send batches. A: Y/N.
However, when it comes to clients... it's much less explicit. Some example questions we could apply:
batch
capability by default?I dunno, it's hard to say when a client really supports 'batches'. What I think makes the most sense is for the actual
batch
column on the standard support page to be green when the client requests / can requestbatch
(and handle messages in batches without dropping them or similar), and to create a similar batch-types support page similar to the SASL mechs support page, detailing exactly which batches are supported.The above is what I'll be going by for the future, and I'll make that batch types support page sometime soon.
The text was updated successfully, but these errors were encountered: