You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/10/16 13:21:35 UTC

[GitHub] [spark] AngersZhuuuu opened a new pull request #26141: [SPARK-29492][SQL]Reset HiveSession's SessionState conf's ClassLoader when sync mode

AngersZhuuuu opened a new pull request #26141: [SPARK-29492][SQL]Reset HiveSession's SessionState conf's ClassLoader when sync mode
URL: https://github.com/apache/spark/pull/26141
 
 
   ### What changes were proposed in this pull request?
   When we use pyhive connect to SparkThriftServer, it will run statement in sync mode.
   When we query data of hive table , it will check serde class in HiveTableScanExec#addColumnMetadataToConf
   
   https://github.com/apache/spark/blob/5a482e72091c8db940408905e8c044f7f5d7814f/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveTableScanExec.scala#L123
   
   Since we run statement in sync mode, it will use HiveSession's SessionState,  and use it's conf's classLoader. then error happened.
   We should reset it when we start run sql in sync mode.
   ### Why are the changes needed?
   Fix bug
   
   ### Does this PR introduce any user-facing change?
   NO
   
   
   ### How was this patch tested?
   UT

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org