Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support dynamic update HeartbeatThread related propertykey #16733

Closed
maobaolong opened this issue Jan 9, 2023 · 4 comments · Fixed by #16702
Closed

Support dynamic update HeartbeatThread related propertykey #16733

maobaolong opened this issue Jan 9, 2023 · 4 comments · Fixed by #16702
Labels
area-configuration type-feature This issue is a feature request

Comments

@maobaolong
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Now, there are over 20 propertykey related HeartbeatThread, which we want to update dynamically.

Describe the solution you'd like

Add listener to HeartbeatThread.

Urgency

Middle

Additional context
No

@LuQQiu
Copy link
Contributor

LuQQiu commented Jan 9, 2023

@maobaolong can you give an example of what's the case you want to update heartbeat thread dynamically?

@maobaolong
Copy link
Contributor Author

maobaolong commented Jan 10, 2023

@LuQQiu For example ttl check interval

Reference this

image

@LuQQiu
Copy link
Contributor

LuQQiu commented Jan 10, 2023

so basically we want the Alluxio cluster maintainer to be able to trigger the job that they want at any given time?

@maobaolong
Copy link
Contributor Author

@LuQQiu Yeah, if the initial heartbeat related interval configures are not good enough, cluster maintainer will use bin/alluxio updateConf command to update the interval propertykey, then the heartbeat interval will changed, this is what we expect.

alluxio-bot pushed a commit that referenced this issue Feb 3, 2023
Fix #16733

pr-link: #16702
change-id: cid-500283d523f5cc5835cb889b85cf7804842a4af6
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this issue May 16, 2023
…HeartbeatThread tick interval

Fix Alluxio#16733

pr-link: Alluxio#16702
change-id: cid-500283d523f5cc5835cb889b85cf7804842a4af6
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this issue May 16, 2023
…HeartbeatThread tick interval

Fix Alluxio#16733

pr-link: Alluxio#16702
change-id: cid-500283d523f5cc5835cb889b85cf7804842a4af6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-configuration type-feature This issue is a feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants