Skip to content

Conversation

Z1Wu
Copy link
Contributor

@Z1Wu Z1Wu commented Sep 3, 2025

Why are the changes needed?

fix issue #7188

How was this patch tested?

ut

Was this patch authored or co-authored using generative AI tooling?

no

…HADOOP_TOKEN_FILE_LOCATION env for SparkProcessBuilder.
@Z1Wu Z1Wu force-pushed the feat/external_token_file_support branch from ea1d2ea to 5712f18 Compare September 3, 2025 15:39
@github-actions github-actions bot added the kind:documentation Documentation is a feature! label Sep 4, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (ea75fa8) to head (f769048).

Files with missing lines Patch % Lines
...bi/credentials/YarnRMDelegationTokenProvider.scala 0.00% 24 Missing ⚠️
...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala 0.00% 13 Missing ⚠️
...ache/kyuubi/engine/spark/SparkProcessBuilder.scala 0.00% 11 Missing ⚠️
...in/scala/org/apache/kyuubi/config/KyuubiConf.scala 0.00% 5 Missing ⚠️
...ache/kyuubi/engine/flink/FlinkProcessBuilder.scala 0.00% 2 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #7189   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         695     696    +1     
  Lines       43433   43472   +39     
  Branches     5887    5889    +2     
======================================
- Misses      43433   43472   +39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants