alex-plekhanov commented on a change in pull request #8960:
URL: https://github.com/apache/ignite/pull/8960#discussion_r606944816
##########
File path: modules/core/src/main/java/org/apache/ignite/client/ClientCache.java
##########
@@ -718,4 +723,27 @@
* @return Cursor.
*/
public FieldsQueryCursor<List<?>> query(SqlFieldsQuery qry);
+
+ /**
+ * Registers a {@link CacheEntryListener}. The supplied {@link
CacheEntryListenerConfiguration} is used to
+ * instantiate a listener and apply it to those events specified in the
configuration.
+ * <p>
+ * NOTE: There is no failover in case of client channel failure, this
event should be handled on the user's side.
+ * Listeners will be notified about client disconnect event if they
implement {@link ClientChannelDisconnectListener}
+ * interface.
+ *
+ * @param cacheEntryListenerConfiguration a factory and related
configuration for creating the listener.
+ * @throws IllegalArgumentException is the same
CacheEntryListenerConfiguration is used more than once or
+ * if some unsupported by thin client properties are set.
+ * @see CacheEntryListener
+ */
+ public void registerCacheEntryListener(CacheEntryListenerConfiguration<K,
V> cacheEntryListenerConfiguration);
Review comment:
But why not provide this ability to the user if it's relatively low cost
and no changes needed on server-side? The user can use these methods in thick
client, and having these methods in thin client makes migration to thin client
easier.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
