Skip to content

S3 Repository: Remove client region#22872

Merged
rjernst merged 2 commits intoelastic:5.xfrom
rjernst:remove_s3_client_region
Jan 30, 2017
Merged

S3 Repository: Remove client region#22872
rjernst merged 2 commits intoelastic:5.xfrom
rjernst:remove_s3_client_region

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Jan 30, 2017

The region setting was added for named clients before the decision was
made to remove region altogether. This change removes the setting,
before it has been released, so as not to introduce an already
deprecated setting.

The region setting was added for named clients before the decision was
made to remove region altogether. This change removes the setting,
before it has been released, so as not to introduce an already
deprecated setting.
Copy link
Copy Markdown
Contributor

@dadoonet dadoonet left a comment

Choose a reason for hiding this comment

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

Thanks!

@rjernst rjernst merged commit 587d548 into elastic:5.x Jan 30, 2017
@rjernst rjernst deleted the remove_s3_client_region branch January 30, 2017 21:50
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Aug 1, 2017
These settings are removed in 6.0. This commit removes the corresponding
documentation.

relates elastic#22872
relates elastic#23984
rjernst added a commit that referenced this pull request Aug 1, 2017
These settings are removed in 6.0. This commit removes the corresponding
documentation.

relates #22872
relates #23984
rjernst added a commit that referenced this pull request Aug 1, 2017
These settings are removed in 6.0. This commit removes the corresponding
documentation.

relates #22872
relates #23984
rjernst added a commit that referenced this pull request Aug 1, 2017
These settings are removed in 6.0. This commit removes the corresponding
documentation.

relates #22872
relates #23984
@clintongormley clintongormley added :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs and removed :Plugin Repository S3 labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >non-issue v5.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants