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

Fetcher implementations are made fail safe by catching exceptions #61

Merged
merged 2 commits into from
Dec 29, 2016

Conversation

thammegowda
Copy link
Member

Summary:

  • Catching all exceptions in the FecthFunction
  • FetcherIterator is generalized into StreamTransformer
  • The exceptions from the Fetcher implementations are propagated up until they are caught at one place

--
@karanjeets Please review and merge

Using embedded jetty to serve web resources from src/main/webapp
This server auto starts before the junit tests and stops after the tests
@arelaxend
Copy link

Working!

@karanjeets
Copy link
Member

@thammegowda LGTM. Merging the PR. 👍

@karanjeets karanjeets merged commit 65c7119 into master Dec 29, 2016
buggtb added a commit that referenced this pull request Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants