Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18811
merged to master
---
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 feature
enabled and wishes so, or
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/18811
ok, have done. Thanks @srowen .
---
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 feature
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18811
Ah, I'm sorry I misled you a bit here @zuotingbing . Yes you found another
unused variable, but, it's in code that is copied directly from Hive. I think
we should leave HiveSessionImplwithUGI
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18811
**[Test build #3869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3869/testReport)**
for PR 18811 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18811
**[Test build #3869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3869/testReport)**
for PR 18811 at commit
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/18811
Thanks @srowen , will do.
---
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 feature
enabled and
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18811
That looks removable, but this isn't really worth bothering with or opening
a JIRA. Can you look for other unused log vars?
---
If your project is set up for it, you can reply to this email and
