[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/19169 I think it could have been useful anyway (eg. a user inserting rows through STS who wants to store the name of the user which is connecting to STS and performing the operation), but I agree that these are very corner cases. I am closing this then. Thanks. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/19169 So far, session-specific user management is not part of our plan yet. This API is not useful before it. Could you please close this PR? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/19169 On local I solved the build error by running a `mvn clean`. As pointed out by @maropu , a PR removed the class and then the incremental compilation fails. I am not sure why this is happening on jenkins though... --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19169 It seems the failure is caused by the version mismatch `v1.3` v.s. `v1.4` in lz4-java (recently, I updated in https://github.com/apache/spark/pull/18883). I'm not sure this happens in jenkins though... --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19169 The test failure with -9 in Jenkins is unknown up to my knowledge (although there have been few clues and assumptions so far). It should be not related with it. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19169
the error the build job is failing for is:
```
core/src/main/scala/org/apache/spark/io/CompressionCodec.scala:24: imported
LZ4BlockInputStream' is permanently hidden by definition of object
LZ4BlockInputStream in package io
[warn] import net.jpountz.lz4.{LZ4BlockInputStream, LZ4BlockOutputStream}
[warn] ^
[error]
core/src/main/scala/org/apache/spark/io/CompressionCodec.scala:120: type
mismatch;
[error] found : Boolean
[error] required: net.jpountz.lz4.LZ4FastDecompressor
[error] new LZ4BlockInputStream(s, disableConcatenationOfByteStream)
[error]^
[info] Boolean <: net.jpountz.lz4.LZ4FastDecompressor?
[info] false
[warn] one warning found
[error] one error found
[error] (core/compile:compileIncremental) Compilation failed
```
this is not to be related to this PR. I am checking the reason which
introduced the problem. Any idea?
Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19169 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19169 **[Test build #81580 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81580/testReport)** for PR 19169 at commit [`6b0b6bd`](https://github.com/apache/spark/commit/6b0b6bd9299c637be30407570d410f2134d8a48f). * This patch **fails due to an unknown error code, -9**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class CurrentUser() extends LeafExpression with CodegenFallback ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19169 **[Test build #81581 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81581/testReport)** for PR 19169 at commit [`6b0b6bd`](https://github.com/apache/spark/commit/6b0b6bd9299c637be30407570d410f2134d8a48f). * This patch **fails due to an unknown error code, -9**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class CurrentUser() extends LeafExpression with CodegenFallback ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19169 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81580/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19169 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81581/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19169 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19169 **[Test build #81581 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81581/testReport)** for PR 19169 at commit [`6b0b6bd`](https://github.com/apache/spark/commit/6b0b6bd9299c637be30407570d410f2134d8a48f). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19169 **[Test build #81580 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81580/testReport)** for PR 19169 at commit [`6b0b6bd`](https://github.com/apache/spark/commit/6b0b6bd9299c637be30407570d410f2134d8a48f). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19169 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19169 looks something is gone wrong with Jenkins command .. not sure why but looks we should just say "ok to test"? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/19169 The Python side looks reasonable, maybe @marmbrus can share his view on adding this function in general? In the mean time Jenkins OK to test. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19169: [SPARK-21957][SQL] Add current_user function
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19169 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
