Skip to content

Fixed S3 globbing which could fail in case of more than 1000 keys and some backends#11179

Merged
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
excitoon-favorites:s3globfix
May 25, 2020
Merged

Fixed S3 globbing which could fail in case of more than 1000 keys and some backends#11179
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
excitoon-favorites:s3globfix

Conversation

@excitoon
Copy link
Copy Markdown
Contributor

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fixed S3 globbing which could fail in case of more than 1000 keys and some backends.

@blinkov blinkov added the pr-bugfix Pull request with bugfix, not backported by default label May 25, 2020
assert expected_err_msg in instance.query_and_get_error(query)


def test_s3_glob_scheherazade(cluster):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Need a comment with cultural references. Not everyone is familiar what is "scheherazade". Wikipedia Ok.

@alexey-milovidov alexey-milovidov self-assigned this May 25, 2020
@alexey-milovidov alexey-milovidov merged commit d4acd1a into ClickHouse:master May 25, 2020
KochetovNicolai pushed a commit that referenced this pull request Jun 8, 2020
Fixed S3 globbing which could fail in case of more than 1000 keys and some backends

(cherry picked from commit d4acd1a)
KochetovNicolai pushed a commit that referenced this pull request Jun 8, 2020
Fixed S3 globbing which could fail in case of more than 1000 keys and some backends

(cherry picked from commit d4acd1a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants