amrishlal commented on a change in pull request #6960:
URL: https://github.com/apache/incubator-pinot/pull/6960#discussion_r638092754
##########
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/column/PhysicalColumnIndexContainer.java
##########
@@ -281,17 +281,18 @@ public static BaseImmutableDictionary
loadDictionary(PinotDataBuffer dictionaryB
}
private static ForwardIndexReader<?> loadRawForwardIndex(PinotDataBuffer
forwardIndexBuffer, DataType dataType) {
- switch (dataType.getStoredType()) {
+ DataType storedType = dataType.getStoredType();
+ switch (storedType) {
case INT:
case LONG:
case FLOAT:
case DOUBLE:
- return new FixedByteChunkSVForwardIndexReader(forwardIndexBuffer,
dataType);
+ return new FixedByteChunkSVForwardIndexReader(forwardIndexBuffer,
storedType);
case STRING:
case BYTES:
- return new VarByteChunkSVForwardIndexReader(forwardIndexBuffer,
dataType);
+ return new VarByteChunkSVForwardIndexReader(forwardIndexBuffer,
storedType);
default:
- throw new IllegalStateException("Illegal data type for raw forward
index: " + dataType);
+ throw new IllegalStateException("Illegal data type for raw forward
index: " + storedType);
Review comment:
Done.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
