Hi Matt Ingenthron,
I completely agree with you in terms of deprecation and adding new interfaces, but I just want to clarify a couple of things.
First, the version API was removed even before Analytics Developer Preview 5 (volatile APIs), which was the first ever integrated version of analytics in CB 5.5 release. So, if anything is broken, it has been broken since then. The removal of the API was tracked by
MB-29837. I agree that things would've been easier if this was communicated with a wider audience, but as I said in my previous post, we didn't think anyone would use that API, especially in a volatile APIs state that gives us liberty of changing/removing any API before beta/GA. In addition, the Analytics SDK-RFC that you guys shared with us had no mention of using this API. If the SDKs were using the output of that API, any regression test would've caught it. Do SDKs ensure the returned response code must be (200) to decide on the result of keep-alive? if not, then I think even with the API removed, that might still be working. QE has been using the Java SDK for system/longevity tests on Alice without issues and the only noise right now is that generated log. But I would imagine any supported use case of analytics will be with the Alice release so I would expect users to update the SDKs to the version that is fully compatible with Alice before using them with Analytics. Having said that, I don't think there is a need to revert the removal of that API, since it was removed when the APIs were in a volatile state, unless it is breaking the current/older SDKs functionality in a really bad way.
Second, about the new ping API that was added today. Each ping request generated by the SDK before today was proxied to the analytics master node, which adds an overhead of establishing the proxy connection. The new ping API that was added today eliminates that proxying step and returns the response directly from the contacted node. SDKs will automatically use that newly added API for pings without any changes needed with the Alice release. Unfortunately, we don't have an MB for the addition of the new ping API anymore. I wished that Michael N. just filed a new issue on MB and changed this as a requested change on SDKs with me as the reported instead of moving this one as an MB. Please feel free to move this back as
MB-31375 and make sure it is marked as resolved with Alice as the fix version. Please file the required SDKs changes to replace the use of the removed version API for keep-alive requests by the ping API, similar to QueryHandler.