[GitHub] [ignite] alex-plekhanov commented on a diff in pull request #10649: IGNITE-17345 Thin client: Add affinity hits/misses metric

["alex-plekhanov (via GitHub)" <gi...@apache.org>]

alex-plekhanov commented on code in PR #10649:
URL: https://github.com/apache/ignite/pull/10649#discussion_r1166598125


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/cache/ClientCacheSqlQueryRequest.java:
##########
@@ -31,7 +31,7 @@
  * Sql query request.
  */
 @SuppressWarnings("unchecked")
-public class ClientCacheSqlQueryRequest extends ClientCacheDataRequest implements ClientTxAwareRequest {
+public class ClientCacheSqlQueryRequest extends ClientCacheQueryRequest implements ClientTxAwareRequest {

Review Comment:
   This change is only to build correct hierarchy, it's unnecessory now, but this is query request and it's better to be child of common query parent. 



-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org