Skip to content

[SERVER] Simplify engine discovery polling after submission exits - #7737

Open
ruanwenjun wants to merge 1 commit into
apache:masterfrom
ruanwenjun:improve/engine-discovery-poll-interval
Open

[SERVER] Simplify engine discovery polling after submission exits#7737
ruanwenjun wants to merge 1 commit into
apache:masterfrom
ruanwenjun:improve/engine-discovery-poll-interval

Conversation

@ruanwenjun

Copy link
Copy Markdown
Member

Why are the changes needed?

Simplify engine discovery polling by moving the one-second delay outside the application-manager callback after a successful submission process exit. This removes the previous-application-info guard and its unused state. The existing process wait remains unchanged while submission is running.

How was this patch tested?

  • build/mvn test-compile -pl kyuubi-server -am -Pspark-provided,flink-provided,hive-provided
  • Temporary local checks verified polling intervals with no application manager and with unavailable application information; they fail before the change and pass afterward. These checks are not included in the PR.
  • dev/reformat and git diff --check.

Was this patch assisted by generative AI tooling?

Assisted-by: OpenAI Codex (GPT-6)

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.

1 participant