alex-plekhanov commented on code in PR #11176:
URL: https://github.com/apache/ignite/pull/11176#discussion_r1481508311
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/cache/ClientCacheScanQueryRequest.java:
##########
@@ -74,7 +74,8 @@ public ClientCacheScanQueryRequest(BinaryRawReaderEx reader) {
/** {@inheritDoc} */
@Override public ClientResponse process(ClientConnectionContext ctx) {
- IgniteCache cache = filterPlatform == ClientPlatform.JAVA &&
!isKeepBinary() ? rawCache(ctx) : cache(ctx);
+ IgniteCache<Object, Object> cache = filterPlatform ==
ClientPlatform.JAVA && !isKeepBinary() ?
+ rawCache(ctx) : cache(ctx);
Review Comment:
It's not specified by our code style and is a matter of taste. For Ignite is
more common to use `?` at the end of the line. For example, I found 375
occurances of `\?$` pattern in .java files of ignite-core module versus 213
occurances of `^\s+\?`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
