Skip to content

[Amoro-3487] fix 404 when access hive-upgrade page#3492

Closed
Liammcl wants to merge 2 commits intoapache:masterfrom
Liammcl:master
Closed

[Amoro-3487] fix 404 when access hive-upgrade page#3492
Liammcl wants to merge 2 commits intoapache:masterfrom
Liammcl:master

Conversation

@Liammcl
Copy link
Contributor

@Liammcl Liammcl commented Mar 31, 2025

Why are the changes needed?

Close #3487 .

Brief change log

  1. Changed API request paths to start with a leading / (e.g., api/ams/v1/ to /api/ams/v1/) to ensure consistency and adhere to standardized path conventions.
  2. Functions affected: getUpgradeStatus, getHiveTableDetail, upgradeHiveTable, getUpgradeProperties.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible
  • Add screenshots for manual tests if appropriate
  • Run test locally before making a pull

Documentation

  • Does this pull request introduce a new feature? (yes / no) no
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions bot added the module:ams-dashboard Ams dashboard module label Mar 31, 2025
@Liammcl Liammcl closed this Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-dashboard Ams dashboard module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: got 404 when access hive-upgrade page

1 participant