sashapolo commented on code in PR #2150:
URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1222515812
##########
modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuplePrefix.java:
##########
@@ -75,17 +71,17 @@ public static BinaryTuplePrefix fromBinaryTuple(BinaryTuple
tuple) {
prefixBuffer.put(0, (byte) (flags | PREFIX_FLAG));
- return new BinaryTuplePrefix(tuple.schema(), prefixBuffer);
+ return new BinaryTuplePrefix(tuple.elementCount(), prefixBuffer);
}
@Override
- public int count() {
- return elementCount();
+ public BigDecimal decimalValue(int col) {
+ throw new UnsupportedOperationException("Must never be called, use
BinaryTupleSchema#decimalValue instead");
}
@Override
- public BigDecimal decimalValue(int index) {
- return decimalValue(index, schema.element(index).decimalScale);
+ public int count() {
+ return elementCount();
Review Comment:
Fixed
--
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]
