[Coordinator throttling] Changes in DiscoveryNodeManager to consume the worker load metrics#25688
[Coordinator throttling] Changes in DiscoveryNodeManager to consume the worker load metrics#25688prashantgolash wants to merge 1 commit into
Conversation
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
b2a7c2a to
53ad072
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
53ad072 to
486bc3f
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Pull Request resolved: prestodb#25688 Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
486bc3f to
f53671f
Compare
|
|
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Pull Request resolved: prestodb#25688 Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
f53671f to
c44452f
Compare
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
c44452f to
4698957
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Pull Request resolved: prestodb#25688 Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
4698957 to
d8039b5
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
d8039b5 to
0231836
Compare
…he worker load metrics (prestodb#25688) Summary: Pull Request resolved: prestodb#25688 Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new structure 3. Exposing a function to fetch worker load metrics which we will be used to make scheduling decisions (In future we can clean up the code fro NodeState fetching) Differential Revision: D76911853
ddc7c04 to
4448eb0
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
4448eb0 to
3184472
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
3184472 to
e5f2ef2
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
e5f2ef2 to
cdd5673
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
cdd5673 to
4576914
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
4576914 to
15d8571
Compare
…he worker load metrics (prestodb#25688) Summary: Pull Request resolved: prestodb#25688 Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
15d8571 to
b63b7db
Compare
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
b63b7db to
88609a4
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
88609a4 to
cd06859
Compare
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
…he worker load metrics (prestodb#25688) Summary: Changes for DiscoveryManager to use the new end point added in the underlying diff Logic 1. Boiler plate code for fetching / refreshing the node stats similar to NodeState -> NodeStats. 2. Consuming NodeState from the new response Once we clean up the NodeState endpoint, we can get rid of `RemoteNodeState` and `HttpRemoteNodeState`. Differential Revision: D76911853
|
This pull request was exported from Phabricator. Differential Revision: D76911853 |
Summary:
Changes for DiscoveryManager to use the new end point i.e
/v1/info/statsLogic
In the follow up PRs we can deprecate
/v1/info/state endpointwe can rid ofRemoteNodeStateandHttpRemoteNodeStateDifferential Revision: D76911853