Skip to content

Add EventLogURI to commandContext and update Spark event log directory handling#99

Merged
sanketjadhavSF merged 1 commit intomainfrom
sanketjadhav/change-eventlog-dir
Mar 19, 2026
Merged

Add EventLogURI to commandContext and update Spark event log directory handling#99
sanketjadhavSF merged 1 commit intomainfrom
sanketjadhav/change-eventlog-dir

Conversation

@sanketjadhavSF
Copy link
Copy Markdown
Contributor

@sanketjadhavSF sanketjadhavSF commented Mar 19, 2026

Description:
Currently we put all spark job logs inside separate directories for respective {job_id} due to this Spark History Server does not able to read event logs and it requires all the logs consolidated into single directory in order to render all the spark applications.

This pull request updates how the Spark event log directory is configured for Spark History Server in the sparkeks command implementation. The main change is to use the new EventLogURI field in the commandContext struct instead of the previous logURI field.

Configuration improvements:

  • Added EventLogURI field to the commandContext struct to explicitly specify the event log URI for Spark jobs.
  • Updated logic in applySparkOperatorConfig to use commandContext.EventLogURI for setting the Spark event log directory, replacing the previous use of logURI.

Copilot AI review requested due to automatic review settings March 19, 2026 12:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Spark event log directory configuration in the sparkeks command so Spark History Server can read consolidated event logs, by introducing an explicit EventLogURI in commandContext.

Changes:

  • Added EventLogURI to commandContext for explicitly configuring the Spark event log base URI.
  • Updated applySparkOperatorConfig to set spark.eventLog.dir from commandContext.EventLogURI instead of execCtx.logURI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@sanketjadhavSF sanketjadhavSF merged commit 78091dd into main Mar 19, 2026
11 checks passed
@sanketjadhavSF sanketjadhavSF deleted the sanketjadhav/change-eventlog-dir branch March 19, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants