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

Refresh elasticsearch nodes periodically #1663

Closed
wants to merge 2 commits into from

Conversation

@martint
Copy link
Member

commented Oct 4, 2019

No description provided.

@cla-bot cla-bot bot added the cla-signed label Oct 4, 2019
@martint martint requested review from dain and zhenxiao Oct 4, 2019
Copy link
Member

left a comment

Some minor comments, but otherwise looks good

@dain
dain approved these changes Oct 4, 2019
@martint martint added this to the 320 milestone Oct 8, 2019
martint added 2 commits Oct 4, 2019
This table displays the list of available Elasticsearch nodes
as seen by each of the Presto workers.
@martint martint force-pushed the martint:elastic-nodes-table branch from 25b91b6 to e76704b Oct 8, 2019
@martint martint closed this Oct 8, 2019
@martint martint deleted the martint:elastic-nodes-table branch Oct 8, 2019
@martint

This comment has been minimized.

Copy link
Member Author

commented Oct 8, 2019

The first two commits were merged in release 320, but I forgot to push the last two commits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.