Skip to content

[SPARK-26756][SQL] Support session conf for thriftserver#23680

Closed
caneGuy wants to merge 1 commit intoapache:masterfrom
caneGuy:zhoukang/add-session-conf
Closed

[SPARK-26756][SQL] Support session conf for thriftserver#23680
caneGuy wants to merge 1 commit intoapache:masterfrom
caneGuy:zhoukang/add-session-conf

Conversation

@caneGuy
Copy link
Contributor

@caneGuy caneGuy commented Jan 29, 2019

What changes were proposed in this pull request?

We can add support for session conf.like:
set spark.sql.xxx.xxx=xxx
which can be useful for users.

How was this patch tested?

Exist UT

@SparkQA
Copy link

SparkQA commented Jan 29, 2019

Test build #101789 has finished for PR 23680 at commit 5090e9f.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@caneGuy
Copy link
Contributor Author

caneGuy commented Feb 15, 2019

Could you help review this? @maropu thanks

@nrchakradhar
Copy link

Can you please provide an example of what kind of parameters are to be used here. May be documentation update is also needed.

@maropu
Copy link
Member

maropu commented Feb 17, 2019

Yea, how do we set the parameters in JDBC?

@caneGuy
Copy link
Contributor Author

caneGuy commented Mar 1, 2019

@maropu @nrchakradhar Thanks!
Config start with 'spark.sql' and which is non static can be set by execute command like below:
set spark.sql.xxx.xxx=xxx
I will update the documentation later.

@HyukjinKwon
Copy link
Member

?? do you mean we cannot set the configuration by set ... via Spark thriftserver if we use beeline?

@HyukjinKwon
Copy link
Member

Can you provide reproducible steps?

@caneGuy
Copy link
Contributor Author

caneGuy commented Mar 26, 2019

Thanks @HyukjinKwon i will try to reproduce later.Since we may change some logic of STS

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@HyukjinKwon
Copy link
Member

Closing this due to author's inactivity.

@caneGuy caneGuy deleted the zhoukang/add-session-conf branch June 1, 2020 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants