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

SQL: Change the rolled over file name to consider the compression #76866

Merged
merged 3 commits into from
Aug 24, 2021

Conversation

astefan
Copy link
Contributor

@astefan astefan commented Aug 24, 2021

Following the change in #64472, the rolled over audit file is now compressed. The IT tests that look at the audit file content should account for this change.

Fixes #73657.

@astefan astefan added >test Issues or PRs that are addressing/adding tests :Analytics/SQL SQL querying v8.0.0 labels Aug 24, 2021
@astefan astefan requested review from Luegg and bpintea August 24, 2021 11:24
@elasticmachine elasticmachine added the Team:QL (Deprecated) Meta label for query languages team label Aug 24, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-ql (Team:QL)

@astefan astefan added v7.16.0 v7.15.1 auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) and removed auto-backport Automatically create backport pull requests when merged v7.16.0 v7.15.1 labels Aug 24, 2021
@elasticsearchmachine elasticsearchmachine merged commit b85d687 into elastic:master Aug 24, 2021
Copy link
Contributor

@bpintea bpintea left a comment

Choose a reason for hiding this comment

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

Post mergum LGTM. Thanks for taking care of this.

@costin
Copy link
Member

costin commented Aug 26, 2021

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/SQL SQL querying auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:QL (Deprecated) Meta label for query languages team >test Issues or PRs that are addressing/adding tests v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] JdbcSecurityIT testQueryDocumentExcluded failing
6 participants