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.

Sign up for GitHub

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

Jump to bottom

allow multiple hosts (for resilience/load balancing) in es.resource #74

Closed
costin opened this issue Aug 15, 2013 · 2 comments
Closed

allow multiple hosts (for resilience/load balancing) in es.resource #74

costin opened this issue Aug 15, 2013 · 2 comments
Labels
v1.3.0.M2

Comments

@costin
Copy link
Member

costin commented Aug 15, 2013

Currently we accept only one host but we should allow the user to specify multiple so we can fallback in case one of the nodes goes down.
@ppat
Copy link

ppat commented Sep 14, 2013

If there are different ESTap instances in a multi-flow Cascade, if we can make each of them use different hosts (just to distribute the load), it would be nice.

@costin
Copy link
Member Author

costin commented Nov 2, 2013

The concurrent/load-balancing part has been addressed through #106

costin added a commit that referenced this issue Dec 5, 2013
@costin
initial stab at network refactoring
5ebf3cc 
break RestClient into Transport and Retry/NetworkClient
make it easy to add other transports w/o duplicating retry logic
add fall-back to other nodes if the current one is dead
relates to #74
relates to #114
costin added a commit that referenced this issue Dec 5, 2013
@costin
propagate network refactoring
f0723e4 
isolate commons http dependency in its dedicated package
introduce HttpStatus class
introduce status check methods on Response interface
relates to #114
relates to #74
costin added a commit that referenced this issue Dec 5, 2013
@costin
wrap up network refactor
1a72bc6 
complete code changes and add minor fixes
remove use of array in favor of Map inside HttpStatus
better flow for catching network errors
add discovery of nodes to ES [Input|Output]Format
relates to #114
relates to #74
@costin costin closed this as completed in 4b1afee Dec 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
v1.3.0.M2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@costin @ppat