Copilot commented on code in PR #7319:
URL: https://github.com/apache/ignite-3/pull/7319#discussion_r2671419771
##########
modules/binary-tuple/src/main/java/org/apache/ignite/internal/binarytuple/BinaryTupleReader.java:
##########
@@ -96,9 +98,9 @@ public boolean hasNullValue(int index) {
* @param index Element index.
* @return Element value.
*/
+ @Override
public boolean booleanValue(int index) {
seek(index);
-
return booleanValue(byteBufferAccessor, begin, end);
Review Comment:
Minor formatting inconsistency: The blank line after `seek(index);` was
removed in the primitive value method, but is still present in the boxed value
method below. Consider keeping the formatting consistent between these methods
for better code readability.
--
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]
