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

Fix the type for LROPoller wait timeout #21913

Merged
merged 1 commit into from
Nov 29, 2021
Merged

Conversation

akx
Copy link
Contributor

@akx akx commented Nov 29, 2021

Thread.join(), as called by wait(), accepts fractional seconds, so the type for wait() should reflect that.

This is a forward-port of Azure/msrest-for-python#242, as @lmazuel kindly pointed out that repo is no longer in active use.

Thread.join(), as called by wait(), accepts fractional seconds,
so the type for wait() should reflect that.
@ghost ghost added Azure.Core customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Nov 29, 2021
@ghost
Copy link

ghost commented Nov 29, 2021

Thank you for your contribution akx! We will review the pull request and get back to you soon.

@xiangyan99
Copy link
Member

/check-enforcer override

@xiangyan99 xiangyan99 merged commit 6bcbe7f into Azure:main Nov 29, 2021
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Dec 1, 2021
…into add_body_error

* 'main' of https://github.com/Azure/azure-sdk-for-python: (74 commits)
  Pin Crypto Temporarily (Azure#21950)
  [release-helper] check `default tag` and `target tag` consistency (Azure#21952)
  [EventHubs] remove unused import (Azure#21949)
  [KV] Use shared test-resources.json (Azure#21940)
  [EventHubs] Fix pylint and mypy (Azure#21939)
  fix line in general document sample code (Azure#21936)
  Pipeline auto-discover stress test (Azure#21917)
  resolve the issue the bad condition in dump proxy logs (Azure#21947)
  [KV] Correctly fetch private RSA key material (Azure#21938)
  doc (Azure#21830)
  Sync eng/common directory with azure-sdk-tools for PR 2348 (Azure#21919)
  [EventHubs] ensure ownership arg does not mutate (Azure#20008)
  Fix the type for LROPoller wait timeout (Azure#21913)
  [EventHubs]Update async samples (Azure#21857)
  [AutoRelease] t2-containerservice-2021-11-25-33508 (Azure#21891)
  update for Go (Azure#21898)
  [pipeline] SDK Release helper : 0.0.1 (Azure#21888)
  [AutoRelease] t2-subscription-2021-11-25-24271 (Azure#21892)
  Release issue status (Azure#21887)
  [AutoRelease] t2-datafactory-2021-11-20-87712 (Azure#21840)
  ...
hildurhodd pushed a commit to hildurhodd/azure-sdk-for-python that referenced this pull request Jan 4, 2022
Thread.join(), as called by wait(), accepts fractional seconds,
so the type for wait() should reflect that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core customer-reported Issues that are reported by GitHub users external to the Azure organization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants