LuciferYang closed pull request #50178: [SPARK-51365][TESTS] Test maven + macos
URL: https://github.com/apache/spark/pull/50178
--
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.
LuciferYang commented on code in PR #50178:
URL: https://github.com/apache/spark/pull/50178#discussion_r1982726209
##
sql/core/src/test/scala/org/apache/spark/sql/test/SharedSparkSession.scala:
##
@@ -79,6 +79,8 @@ trait SharedSparkSessionBase
StaticSQLConf.WAREHOUSE_PATH
LuciferYang commented on PR #50178:
URL: https://github.com/apache/spark/pull/50178#issuecomment-2705877001
> The issue should be resolvable. The commit history of this pr is a bit
messy. I'll submit a clean one and add more descriptions tomorrow
a clean one: https://github.com/apache
LuciferYang commented on PR #50178:
URL: https://github.com/apache/spark/pull/50178#issuecomment-2704523092
The issue should be resolvable.
The commit history of this pr is a bit messy. I'll submit a clean one and
add more descriptions tomorrow
--
This is an automated message from the
LuciferYang commented on code in PR #50178:
URL: https://github.com/apache/spark/pull/50178#discussion_r1982726565
##
.github/workflows/maven_test.yml:
##
@@ -184,6 +184,7 @@ jobs:
- name: Run tests
env: ${{ fromJSON(inputs.envs) }}
run: |
+ uli
dongjoon-hyun commented on code in PR #50178:
URL: https://github.com/apache/spark/pull/50178#discussion_r1982774552
##
sql/core/src/test/scala/org/apache/spark/sql/test/SharedSparkSession.scala:
##
@@ -79,6 +79,8 @@ trait SharedSparkSessionBase
StaticSQLConf.WAREHOUSE_PA
LuciferYang commented on code in PR #50178:
URL: https://github.com/apache/spark/pull/50178#discussion_r1982784603
##
sql/hive/src/test/scala/org/apache/spark/sql/hive/test/TestHive.scala:
##
@@ -44,33 +44,48 @@ import
org.apache.spark.sql.execution.{CommandExecutionMode, Query
LuciferYang opened a new pull request, #50178:
URL: https://github.com/apache/spark/pull/50178
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How
