[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5645: Fix ORC Record reader to ignore extra fields

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on a change in pull request #5645:
URL: https://github.com/apache/incubator-pinot/pull/5645#discussion_r448521328



##########
File path: pinot-plugins/pinot-input-format/pinot-orc/src/main/java/org/apache/pinot/plugin/inputformat/orc/ORCRecordReader.java
##########
@@ -324,7 +324,7 @@ private static Object extractSingleValue(ColumnVector columnVector, int rowId, T
         }
       default:
         // Unsupported types
-        throw new IllegalStateException("Unsupported field type: " + category);
+        throw new IllegalStateException("Unsupported field type: " + category + " for field " + field);

Review comment:
       (nit)
   ```suggestion
           throw new IllegalStateException("Unsupported field type: " + category + " for field: " + field);
   ```




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org