marton-bod commented on code in PR #3131:
URL: https://github.com/apache/hive/pull/3131#discussion_r847339367
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/MapOperator.java:
##########
@@ -673,7 +674,31 @@ private String toErrorMessage(Writable value, Object row,
ObjectInspector inspec
ctx.getIoCxt().setRecordIdentifier(null);//so we don't
accidentally cache the value; shouldn't
//happen since IO layer either knows how to produce ROW__ID or not
- but to be safe
}
- break;
+ break;
+ case PARTITION_SPEC_ID:
Review Comment:
Unfortunately we don't have the Table object anywhere around this area as
far as I can tell, so I'm not sure how we could inject the logic using the
storage handler. Besides, this method is called `populateVirtualColumns` where
all the other virtual cols are filled out too, so right now I don't see a
better place to put it
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
