GitHub user ChenjunZou opened a pull request:
https://github.com/apache/spark/pull/19961
[SPARK-22496][SQL] thrift server adds operation logs
## What changes were proposed in this pull request?
since hive 2.0+ upgrades log4j to log4j2ï¼a lot of
[changes](https://issues.apache.org/jira/browse/HIVE-11304) are made working on
it.
as spark is not to ready to update its inner hive version(1.2.1) , so I
manage to make little changes.
the function registerCurrentOperationLog is moved from SQLOperstion to its
parent class ExecuteStatementOperation so spark can use it.
## How was this patch tested?
manual test
Author: zouchenjun <zouchen...@youzan.com>
Closes #19721 from ChenjunZou/operation-log.
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration
tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise,
remove this)
Please review http://spark.apache.org/contributing.html before opening a
pull request.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ChenjunZou/spark spark-22496
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19961.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #19961
----
commit b39d36fba5ffbbcac6ba9c15b4062b7ade7b6f1e
Author: zouchenjun <zouchen...@youzan.com>
Date: 2017-12-11T04:36:14Z
[SPARK-22496][SQL] thrift server adds operation logs
## What changes were proposed in this pull request?
since hive 2.0+ upgrades log4j to log4j2ï¼a lot of
[changes](https://issues.apache.org/jira/browse/HIVE-11304) are made working on
it.
as spark is not to ready to update its inner hive version(1.2.1) , so I
manage to make little changes.
the function registerCurrentOperationLog is moved from SQLOperstion to its
parent class ExecuteStatementOperation so spark can use it.
## How was this patch tested?
manual test
Author: zouchenjun <zouchen...@youzan.com>
Closes #19721 from ChenjunZou/operation-log.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
