Sunjeet

Thanks for this contribution. Changing the return type of the `triggerAsyncRefresh` method from `void` to `CompletableFuture` would cause binary backwards incompatibility. Is there a reason to not simply use the...

NOTE: we also saw in a list with no ordering changes, a diff was presented without any actual differences. Could be related.

@yonatang your proposed tweak sounds good to me [EDIT: I take my above approval back, thanks Nayanika for pointing out to me that if a refresh listener throws an exception...

Upon a closer look, @yonatang it might not be a good idea to log those metrics using a `RefreshListener`. Basically, regardless of whether we update those metrics before or after...

> When are you planning to release this pr ? Hi @mailsanchu no plans atm. There are a few gaps like handling edge cases, poor performance, and language support for...

Wouldn't having one producer per dataset and using the producer's `restore` achieve this?