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

Fix HDFS tests with Camel 3.4 (related to issue #263) #309

Merged
merged 1 commit into from
Jun 30, 2020

Conversation

orpiske
Copy link
Contributor

@orpiske orpiske commented Jun 30, 2020

No description provided.

@orpiske
Copy link
Contributor Author

orpiske commented Jun 30, 2020

Strange error. It's unrelated to the change and I cannot reproduce it locally. Any ideas @oscerd ?

@oscerd
Copy link
Contributor

oscerd commented Jun 30, 2020

I'm re-running the test. No idea really.

@oscerd
Copy link
Contributor

oscerd commented Jun 30, 2020

Maybe we could remove the workaround

@orpiske
Copy link
Contributor Author

orpiske commented Jun 30, 2020

Maybe we could remove the workaround

+1. I logged an issue (#310) for this just in case we want to hold a bit on it.

@oscerd
Copy link
Contributor

oscerd commented Jun 30, 2020

The containerlifecycle error seems to be related to the cometd version used aganist Jetty9

@orpiske
Copy link
Contributor Author

orpiske commented Jun 30, 2020

The containerlifecycle error seems to be related to the cometd version used aganist Jetty9

Great catch, thanks! I am on it.

@orpiske
Copy link
Contributor Author

orpiske commented Jun 30, 2020

That did the trick, but I'll squash it so we have a clean history.

@orpiske orpiske force-pushed the fix-hdfs-deps-camel-3.4 branch from 5b07e32 to b64b356 Compare June 30, 2020 10:50
@oscerd oscerd merged commit 6acdab1 into apache:master Jun 30, 2020
@oscerd
Copy link
Contributor

oscerd commented Jun 30, 2020

Squashed and merged.

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.

2 participants