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: bump github.com/Azure/azure-kusto-go from 0.5.0 to 0.60 and make timing more granular #10966

Merged
merged 6 commits into from
Apr 12, 2022

Conversation

AsafMah
Copy link
Contributor

@AsafMah AsafMah commented Apr 12, 2022

Required for all PRs:

  • Updated associated README.md - not needed, only an internal change.
  • Wrote appropriate unit tests - not needed, no new functionality.
  • Pull request title or commits are in conventional commit format

Version 0.6.0 of the go SDK, among other irrelevant changes, updates its azure blob library to a version that doesn't leak memory and solves upload failures.

Additionally, when serializing the timestamp to kusto, the granularity was increased from a second to a nanosecond, and the format is now hardcoded to be time.RFC3339Nano, which kusto accepts.

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Apr 12, 2022
@telegraf-tiger
Copy link
Contributor

@powersj powersj added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Apr 12, 2022
@powersj powersj merged commit 47113ef into influxdata:master Apr 12, 2022
sspaink pushed a commit that referenced this pull request Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants