[GitHub] [linkis] CharlieYan24 commented on a diff in pull request #4207: fix: HIVE job log contains 'java.lang.NullPointerException' error.(#4206)

["CharlieYan24 (via GitHub)" <gi...@apache.org>]

CharlieYan24 commented on code in PR #4207:
URL: https://github.com/apache/linkis/pull/4207#discussion_r1103902587


##########
linkis-engineconn-plugins/hive/src/main/scala/org/apache/linkis/engineplugin/hive/executor/HiveEngineConnExecutor.scala:
##########
@@ -205,7 +205,15 @@ class HiveEngineConnExecutor(
       val startTime = System.currentTimeMillis()
       try {
         Utils.tryCatch {
-          val queryPlan = driver.getPlan
+          var queryPlan = driver.getPlan

Review Comment:
   > This problem is caused by another fix. It is recommended to optimize and add parameters, whether to enable compilation.#3906
   > 
   > ```
   > eg:
   > val ENABLE_HIVE_COMPILE =
   >     CommonVars[Boolean]("linkis.hive.enable.compile", true).getValue
   > ```
   > 
   > ![image](https://user-images.githubusercontent.com/11496700/218317978-671c0b4c-5ad8-4183-af26-f3b2d45b7f7f.png)
   
   Thanks for your kind advice. I will optimize and add parameters, whether to enable compilation.



-- 
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: notifications-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org