vineetgarg02 commented on a change in pull request #1472:
URL: https://github.com/apache/hive/pull/1472#discussion_r492224561
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java
##########
@@ -63,19 +63,19 @@
private VectorizationContext taskVectorizationContext;
- protected transient JobConf jc;
- private transient boolean inputFileChanged = false;
+ protected JobConf jc;
Review comment:
I actually tried not keeping these fields but I was running into all
sorts of issues like unable to serialize/de-serialize or plan generating
without metadata etc.
I am not sure if we need to keep all of these fields or we can selectively
choose, I went by almost all in interest of time. If Gopal or Rajesh thinks
that this may cause performance issue I can open a follow-up to investigate and
choose fields selectively.
----------------------------------------------------------------
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]
