Skip to content

Conversation

sbiscigl
Copy link
Contributor

Issue #, if available:

#3568

Description of changes:

Very similar to a .NET issue aws/aws-sdk-net#3349 the we also have where we attempt endpoint discovery when the endpoint is overridden. This is not the desired behavior we want to use the overridden endpoint skipping discovery all together. this aligns with the .NET solution.

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sbiscigl sbiscigl force-pushed the timestream-query-ep-discovery branch 2 times, most recently from 39a230a to 1e92e9c Compare September 30, 2025 12:53
@sbiscigl sbiscigl marked this pull request as ready for review September 30, 2025 13:40
@sbiscigl sbiscigl closed this Sep 30, 2025
@sbiscigl sbiscigl reopened this Sep 30, 2025
@sbiscigl sbiscigl marked this pull request as draft September 30, 2025 13:42
@sbiscigl sbiscigl marked this pull request as ready for review September 30, 2025 13:42
@sbiscigl sbiscigl force-pushed the timestream-query-ep-discovery branch from 1e92e9c to 229c8ea Compare September 30, 2025 15:26
@sbiscigl sbiscigl merged commit c2b1f75 into main Sep 30, 2025
6 checks passed
@sbiscigl sbiscigl deleted the timestream-query-ep-discovery branch September 30, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants