difin commented on code in PR #5538:
URL: https://github.com/apache/hive/pull/5538#discussion_r1870400053
##########
ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java:
##########
@@ -125,6 +130,17 @@ public QueryPlan getPlan() {
public void setPlan(QueryPlan plan) {
this.plan = plan;
+ // only set runtimeContext if the plan is not null
+ // we don't want to nullify runtimeContext if this method is called with
plan=null, which is the case when e.g.
+ // driver.releasePlan() tries to release resources/objects that are known
to be heavy
+ if (plan != null){
Review Comment:
nit: missing space between `){`
--
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]
