ably / docs

Ably Realtime API documentation
https://ably.com/docs
Apache License 2.0
20 stars 41 forks source link

Document transient publishes once more client libraries implement it #357

Open SimonWoolf opened 6 years ago

SimonWoolf commented 6 years ago

currently only implemented in ably-js

┆Issue is synchronized with this Jira Task by Unito

mattheworiordan commented 6 years ago

This is perhaps not something we should do until the next version of our spec is released. Arguably, this is an experimental feature until officially supported.

mattheworiordan commented 6 years ago

Related to https://github.com/ably/docs/issues/319 in regards to how we recommend customers publish large volumes of messages. Closing that issue in favour of this.

mattheworiordan commented 4 years ago

@QuintinWillison I assume we do support transient publishing now in all libs and can close this (if documented)?

QuintinWillison commented 4 years ago

It was documented in 27157ba673aecbf56ee808cbc1b94e38380864a4 but without Java, Objective-C or C# examples. We claim support in both Java and .NET, however I can only find one instance of the word 'transient' in the spec so I'm not entirely clear what I'm looking for.

Java, for example, doesn't have anywhere near the number of members I would expect in its ProtocolMessage.Flag enum.

Before exploring deeper... @mattheworiordan, can you give me any more pointers as to what I should be looking for here? Where is this feature defined?

QuintinWillison commented 4 years ago

ok, as I clunk through my GitHub notifications I see you've indirectly given me the answer with https://github.com/ably/docs/pull/323 ... "published immediately" and "implicit attach" from RTL6 sub-items.

paddybyers commented 4 years ago

The spec change can be found via https://github.com/ably/docs/pulls?utf8=%E2%9C%93&q=is%3Apr+transient+publishing

The java implementation, for example, can be found via https://github.com/ably/ably-java/pulls?utf8=%E2%9C%93&q=is%3Apr+transient+publishing

I thought that we had specified that as a required feature for any library advertising itself as 1.1, so it should be implemented in most or all libraries. Having said that, I don't remember seeing it in PHP or Python.

SimonWoolf commented 4 years ago

Having said that, I don't remember seeing it in PHP or Python.

It's a realtime client lib feature, php and python are currently rest-only

paddybyers commented 4 years ago

It's a realtime client lib feature, php and python are currently rest-only

Yes of course, I was thinking of idempotent publishing for some reason.