[SPARK-3939][SQL] NPE caused by SessionState.out not set in thriftserver2

[adrian-wang]

There would be an NPE caused by SessionState.out not set in thriftserver2.

[SparkQA]

QA tests have started for PR 2812 at commit e89a7c2.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2812 at commit e89a7c2.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21764/
Test FAILed.

[adrian-wang]

retest this please.

[SparkQA]

QA tests have started for PR 2812 at commit e89a7c2.

• This patch merges cleanly.

[shaneknapp]

@adrian-wang i'm gonna kill the currently running test and retrigger it after i do some maintenance on our jenkins infrastructure. cool?

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21773/
Test FAILed.

[shaneknapp]

jenkins, test this please

[SparkQA]

QA tests have started for PR 2812 at commit e89a7c2.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2812 at commit e89a7c2.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21776/
Test FAILed.

[adrian-wang]

retest this please.

[SparkQA]

QA tests have started for PR 2812 at commit e89a7c2.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2812 at commit e89a7c2.

• This patch fails PySpark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21791/
Test FAILed.

[adrian-wang]

retest this please.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21795/
Test FAILed.

[adrian-wang]

retest this please.

[SparkQA]

QA tests have started for PR 2812 at commit e89a7c2.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2812 at commit e89a7c2.

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

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21796/
Test PASSed.

[marmbrus]

Instead of duplicating this code, can we just change the line above to SessionState.start(SparkSQLEnv.hiveContext.sessionState). out and err would be initialized then. I'm actually not sure why we are creating our own session state here at all instead of using the one provided by the context.

[liancheng]

Sorry, didn't notice this PR and also worked on this issue in #2887, which I think is a more fundamental solution.

[adrian-wang]

Never mind, I'll close this then.