Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@sweir-thescore
Copy link

Change Logs

We've been seeing recurring issues with the timeline server being unable to bind a port when running hudi streamer jobs.
This PR copies the changes found in this PR against the main Hudi repo. Please see the change here: https://github.com/apache/hudi/pull/12241/files

Please note that the unit tests created here were not copied because they reference packages that do not exist in our version of Hudi.

Changelog notes from original PR:

  • If the specified port is in use, there is no way for the server to start currently due to a bug in the logic for trying a new port. This PR updates the logic to work properly and adds a test (again, please note, we do not add the unit test here for above reason mentioned)

Impact

Introduces changes from this PR that should help resolve the port retry issue: https://github.com/apache/hudi/pull/12241/files.

Impact noted on the PR is: Fixes existing bug in timeline server code

Risk level (write none, low medium or high below)

Low. If this does not succeed in a lower environment, we can simply revert this change. Additionally, we can also revert it in prod using our regular release/revert process.

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
    ticket number here and follow the instruction to make
    changes to the website.

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@sweir-thescore sweir-thescore requested a review from a team April 28, 2025 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant