szlta commented on a change in pull request #1251:
URL: https://github.com/apache/hive/pull/1251#discussion_r454390429
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
##########
@@ -232,6 +250,17 @@ private VectorizedOrcAcidRowBatchReader(JobConf conf,
OrcSplit orcSplit, Reporte
this.syntheticProps = orcSplit.getSyntheticAcidProps();
+ if (LlapHiveUtils.isLlapMode(conf) && LlapProxy.isDaemon()
+ && HiveConf.getBoolVar(conf, ConfVars.LLAP_TRACK_CACHE_USAGE))
+ {
+ MapWork mapWork = LlapHiveUtils.findMapWork(conf);
Review comment:
We could spare the deserialization of MapWork from JobConf here, if we
pass the MapWork instance already present in LlapRecordReader to
VectorizedOrcAcidRowBatchReader ctor. (Downside is that in turn we would need
to adjust the other ctor's of VectorizedOrcAcidRowBatchReader too)
----------------------------------------------------------------
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]
