Skip to content

Conversation

@imback82
Copy link
Contributor

What changes were proposed in this pull request?

This is a follow up to address #34842 (comment), where setting /tmp as a namespace location may break certain test environments.

This PR also fixes a minor string interpolation issue (unnecessary s) in the same file.

Why are the changes needed?

To fix a test issue that the namespace location may cause in certain environments where /tmp is not accessible.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Updated the test

@github-actions github-actions bot added the SQL label Dec 17, 2021
@imback82
Copy link
Contributor Author

cc @cloud-fan

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50789/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50789/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Test build #146316 has finished for PR 34930 at commit dc63822.

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

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in 6b1c980 Dec 17, 2021
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.

3 participants