[SPARK-35980][CORE] ThreadAudit logs whether thread is daemon

[timarmstrong]

What changes were proposed in this pull request?

Add daemon={true|false} to the POSSIBLE THREAD LEAK IN SUITE warning printed by test framework.

Why are the changes needed?

This is to slightly accelerate interpretation of that warning, since non-daemon threads can block the process from exiting and are likely to be problematic.

Only affects test code.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Manually ran some tests, inspected the output log line.

[HyukjinKwon]

cc @gaborgsomogyi can you take a quick look when you find some time?

[HyukjinKwon]

ok to test

[HyukjinKwon]

Looks fine to me

[SparkQA]

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

[SparkQA]

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

[gaborgsomogyi]

LGTM.

[gaborgsomogyi]

@HyukjinKwon I agree, it's a good addition.

[SparkQA]

Test build #140575 has finished for PR 33178 at commit 6a3ccd1.

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

[gaborgsomogyi]

Seems like unrelated:
java.io.FileNotFoundException: File /home/jenkins/workspace/SparkPullRequestBuilder@2/target/tmp/spark-21674620-ac83-4ad3-a153-5a7adf909244/20.zip does not exist

[gaborgsomogyi]

retest this please

[gaborgsomogyi]

Filed SPARK-35993 to track the issue.

[SparkQA]

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

[SparkQA]

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

[SparkQA]

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

[SparkQA]

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

[SparkQA]

Test build #140584 has finished for PR 33178 at commit 6a3ccd1.

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

[timarmstrong]

Thanks for the quick reviews!

[AmplabJenkins]

Can one of the admins verify this patch?

[HyukjinKwon]

Merged to master.