Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18666
@liufengdb yes you are right. what's more, the Hive's `SessionState` is too
overstaffed for spark to create an hive client, we may only need an
`IMetaStoreClient`
---
--
Github user liufengdb commented on the issue:
https://github.com/apache/spark/pull/18666
Maybe I missed something, but it seems Spark has its own class loader right
now, which can load the class from the given URL:
https://github.com/apache/spark/blob/master/sql/hive/src/main/scala/or
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18666
@liufengdb its necessary to create these for `add jar` cmd
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user liufengdb commented on the issue:
https://github.com/apache/spark/pull/18666
I asked the following question in
https://github.com/apache/spark/pull/20864: is it necessary to create these
temp directories when the hive thrift server starts? It sounds some legacy from
Hive
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1633/
Tes
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18666
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1625/
Tes
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18666
@gatorsmile would you plz take a look at this.
this pr mainly want to close HiveSessionState explicitly to delete
`hive.downloaded.resources.dir` which points to `"${system:java.io.tmpdir}"
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1624/
Tes
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/18666
@samartinucci thanks for reminding of this, i have fixed the conflicts.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user samartinucci commented on the issue:
https://github.com/apache/spark/pull/18666
Do we have a workaround for this? Will this PR get merged?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18666
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
15 matches
Mail list logo