You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2021/08/25 00:15:41 UTC
[GitHub] [helix] xyuanlu opened a new issue #1851: Inconsistent behavior in getIdealAssignmentForFullAuto and getImmediateAssignmentForWagedFullAuto
xyuanlu opened a new issue #1851:
URL: https://github.com/apache/helix/issues/1851
**Is your feature request related to a problem? Please describe.**
Helix has 2 functions to return potential cluster partition assignment for any cluster state change. getIdealAssignmentForFullAuto for ChrushEd resources and getImmediateAssignmentForWagedFullAuto for Waged resources.
When recomputing assignment, getIdealAssignmentForFullAuto will not consider delayed rebalance and return the assignment. However, getImmediateAssignmentForWagedFullAuto will honor the current timestamp and the delayed rebalance setting.
In rest API partitionAssignment is a single enter point for cluster assignment, it called these 2 functions for different resource types. The behavior is inconsistent for these two.
**Describe the solution you'd like**
For now, We are disabling delayed rebalance for partitionAssignment. Could add a cluster option to honor delayed rebalance window in the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org
[GitHub] [helix] jiajunwang commented on issue #1851: Inconsistent behavior in getIdealAssignmentForFullAuto and getImmediateAssignmentForWagedFullAuto -- delayed rebalance
Posted by GitBox <gi...@apache.org>.
jiajunwang commented on issue #1851:
URL: https://github.com/apache/helix/issues/1851#issuecomment-909437512
@xyuanlu this issue has been resolved by the attached PR, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org
[GitHub] [helix] xyuanlu commented on issue #1851: Inconsistent behavior in getIdealAssignmentForFullAuto and getImmediateAssignmentForWagedFullAuto -- delayed rebalance
Posted by GitBox <gi...@apache.org>.
xyuanlu commented on issue #1851:
URL: https://github.com/apache/helix/issues/1851#issuecomment-909689411
> @xyuanlu this issue has been resolved by the attached PR, right?
Not really... these 2 APIs in HelixUtil still have inconsistent behavior. The change in PR only resolve the inconsistent in PartitionAssignment API.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org
[GitHub] [helix] jiajunwang commented on issue #1851: Inconsistent behavior in getIdealAssignmentForFullAuto and getImmediateAssignmentForWagedFullAuto -- delayed rebalance
Posted by GitBox <gi...@apache.org>.
jiajunwang commented on issue #1851:
URL: https://github.com/apache/helix/issues/1851#issuecomment-909437512
@xyuanlu this issue has been resolved by the attached PR, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org
[GitHub] [helix] xyuanlu commented on issue #1851: Inconsistent behavior in getIdealAssignmentForFullAuto and getImmediateAssignmentForWagedFullAuto -- delayed rebalance
Posted by GitBox <gi...@apache.org>.
xyuanlu commented on issue #1851:
URL: https://github.com/apache/helix/issues/1851#issuecomment-909689411
> @xyuanlu this issue has been resolved by the attached PR, right?
Not really... these 2 APIs in HelixUtil still have inconsistent behavior. The change in PR only resolve the inconsistent in PartitionAssignment API.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org