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

CALCITE-4646 Update to Jetty 9.4.42.v20210604 #147

Closed
wants to merge 3 commits into from

Conversation

stoty
Copy link
Contributor

@stoty stoty commented Jun 12, 2021

No description provided.

@stoty
Copy link
Contributor Author

stoty commented Jun 12, 2021

The failing Windows test may be caused by the new Jetty version.
This patch doesn't touch anything in the Avatica codebase that could case that.

@stoty
Copy link
Contributor Author

stoty commented Jun 12, 2021

The test fixes are for removing an illegal leading slash in the file paths on Windows.
These were accepted by earlier Jetty versions, but not by 9.4.42.

…n the test

Change-Id: I04d13b49eba395b2ae41e31b962d1bd4c3ed5cc2
Change-Id: I9d41880c427ce359e3e284f1f94d9aa877d33234
@stoty stoty force-pushed the CALCITE-4646 branch 2 times, most recently from 888df5f to 0294623 Compare July 6, 2021 05:25
Copy link
Member

@zabetak zabetak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, if nobody objects I will merge this PR soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants