Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

[AURON #2011] History Server fails when BuildInfo event is missing.#2012

Open
slfan1989 wants to merge 1 commit intoapache:masterapache/auron:masterfrom
slfan1989:auron-2011slfan1989/auron:auron-2011Copy head branch name to clipboard
Open

[AURON #2011] History Server fails when BuildInfo event is missing.#2012
slfan1989 wants to merge 1 commit intoapache:masterapache/auron:masterfrom
slfan1989:auron-2011slfan1989/auron:auron-2011Copy head branch name to clipboard

Conversation

@slfan1989
Copy link
Contributor

Which issue does this PR close?

Closes #2011

Rationale for this change

The History Server plugin currently crashes during initialization when the AuronBuildInfoUIData record is missing from the KVStore. This causes applications without BuildInfo events to either fail plugin initialization or show no Auron tab.

What changes are included in this PR?

  1. AuronSQLAppStatusStore: Changed buildInfo() to return Option[AuronBuildInfoUIData], catching NoSuchElementException and other exceptions to return None instead of throwing
  2. AuronSQLHistoryServerPlugin: Removed the null check and always create the Auron tab, letting the UI handle empty state
  3. AuronAllExecutionsPage: Added buildInfoSummary() method to handle Option[AuronBuildInfoUIData]:
    • Some: displays BuildInfo table as before
    • None: shows user-friendly message "Auron build information is not available for this application."

Are there any user-facing changes?

Yes. When BuildInfo is not available:

  • Before: Plugin initialization fails or no Auron tab appears
  • After: Auron tab displays with a clear warning message explaining BuildInfo is unavailable

How was this patch tested?

  • Existing unit tests pass

…ing.

Signed-off-by: slfan1989 <slfan1989@apache.org>
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.

History Server fails when BuildInfo event is missing

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.