This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.5 by this push:
     new fde0fe676358 [SPARK-45975][SQL][TESTS][3.5] Reset 
storeAssignmentPolicy to original
fde0fe676358 is described below

commit fde0fe676358fb3e9142d6895e4c7fc2e6604d5e
Author: wforget <643348...@qq.com>
AuthorDate: Sat Dec 2 11:25:54 2023 -0800

    [SPARK-45975][SQL][TESTS][3.5] Reset storeAssignmentPolicy to original
    
    ### What changes were proposed in this pull request?
    Reset storeAssignmentPolicy to original in HiveCompatibilitySuite.
    
    ### Why are the changes needed?
    STORE_ASSIGNMENT_POLICY was not reset in HiveCompatibilitySuite, causing 
subsequent test cases to fail.
    
    Details: 
https://github.com/wForget/spark/actions/runs/6902668865/job/18779862759
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    Pass GitHub Actions
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #44126 from LuciferYang/SPARK-45943-FOLLOWUP.
    
    Authored-by: wforget <643348...@qq.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 .../org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala   | 3 +++
 1 file changed, 3 insertions(+)

diff --git 
a/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
 
b/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
index bd323dc4b24e..0467603c01cd 100644
--- 
a/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
+++ 
b/sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
@@ -41,6 +41,8 @@ class HiveCompatibilitySuite extends HiveQueryFileTest with 
BeforeAndAfter {
   private val originalCrossJoinEnabled = TestHive.conf.crossJoinEnabled
   private val originalSessionLocalTimeZone = TestHive.conf.sessionLocalTimeZone
   private val originalAnsiMode = TestHive.conf.getConf(SQLConf.ANSI_ENABLED)
+  private val originalStoreAssignmentPolicy =
+    TestHive.conf.getConf(SQLConf.STORE_ASSIGNMENT_POLICY)
   private val originalCreateHiveTable =
     TestHive.conf.getConf(SQLConf.LEGACY_CREATE_HIVE_TABLE_BY_DEFAULT)
 
@@ -76,6 +78,7 @@ class HiveCompatibilitySuite extends HiveQueryFileTest with 
BeforeAndAfter {
       TestHive.setConf(SQLConf.CROSS_JOINS_ENABLED, originalCrossJoinEnabled)
       TestHive.setConf(SQLConf.SESSION_LOCAL_TIMEZONE, 
originalSessionLocalTimeZone)
       TestHive.setConf(SQLConf.ANSI_ENABLED, originalAnsiMode)
+      TestHive.setConf(SQLConf.STORE_ASSIGNMENT_POLICY, 
originalStoreAssignmentPolicy)
       TestHive.setConf(SQLConf.LEGACY_CREATE_HIVE_TABLE_BY_DEFAULT, 
originalCreateHiveTable)
 
       // For debugging dump some statistics about how much time was spent in 
various optimizer rules


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to