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

DRILL-8438: Bump YAUAA to 7.19.2 #2808

Merged
merged 2 commits into from
May 23, 2023
Merged

Conversation

nielsbasjes
Copy link
Contributor

Updating Yauaa to latest version + fixing a bug in the caching in a test.
See #2807

@cgivre
I did 2 things here:

  • Update Yauaa and use the built in selection mechanism to the caching.
  • Fix a bug in the test that still used the Client Hints caching.

Let's see what the CI build thinks of this.

@cgivre cgivre changed the title chore(deps): Update Yauaa to 7.19.2 DRILL-8438: Bump YAUAA to 7.19.2 May 23, 2023
@cgivre cgivre self-requested a review May 23, 2023 11:56
Copy link
Contributor

@cgivre cgivre left a comment

Choose a reason for hiding this comment

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

LGTM +1 (Pending CI)

@cgivre
Copy link
Contributor

cgivre commented May 23, 2023

@nielsbasjes Thanks for this. I'll rerun the failed test. Not sure why we're getting network issues but that seems unrelated to this PR.

@nielsbasjes
Copy link
Contributor Author

My fixes seem to have had the desired effect.

@cgivre cgivre merged commit 214e188 into apache:master May 23, 2023
@nielsbasjes nielsbasjes deleted the Update_Yauaa_7.19.2 branch May 23, 2023 23:28
jnturton pushed a commit that referenced this pull request Dec 31, 2023
* chore(deps): Update Yauaa to 7.19.2

* Additional fix, please squash on merge
jnturton pushed a commit to jnturton/drill that referenced this pull request Dec 31, 2023
* chore(deps): Update Yauaa to 7.19.2

* Additional fix, please squash on merge
jnturton pushed a commit to jnturton/drill that referenced this pull request Jan 3, 2024
* chore(deps): Update Yauaa to 7.19.2

* Additional fix, please squash on merge
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.

2 participants