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

Feature/add kusto streaming feature #364

Merged
merged 35 commits into from
Jan 2, 2025

Conversation

ag-ramachandran
Copy link
Contributor

Pull Request Description

This is the PR that was for Kusto Streaming. Cherry-picked from the PR #301 and rebased with master

This adds a new option called Stream, that streams data using ManagedStreamingIngest client to Kusto.

Did a couple of large runs to Kusto . While it seems to run satisfactorily, many times it falls back to queued ingest in case the volumes are high

image

Lower volumes seem to be using streaming quite effectively

image

Future Release Comment

[Add description of your change, to include in the next release]
[Delete any or all irrelevant sections, e.g. if your change does not warrant a release comment at all]

Breaking Changes:

  • None

Features:

  • None

Fixes:

  • None

Copy link
Contributor

@ohadbitt ohadbitt left a comment

Choose a reason for hiding this comment

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

We can schedule a conversation to go over it all :)

docs/KustoSink.md Outdated Show resolved Hide resolved
docs/KustoSink.md Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Feb 28, 2024

Test Results

81 tests  +3   81 ✅ +3   2m 46s ⏱️ -8s
22 suites ±0    0 💤 ±0 
22 files   ±0    0 ❌ ±0 

Results for commit 15222ef. ± Comparison against base commit 7c0b90b.

♻️ This comment has been updated with latest results.

@ag-ramachandran ag-ramachandran force-pushed the feature/AddKustoStreamingFeature branch 5 times, most recently from f44ba1a to 97529c8 Compare March 6, 2024 08:58
Copy link
Contributor

@ohadbitt ohadbitt left a comment

Choose a reason for hiding this comment

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

We still have some TODOs we left here - just make sure we're ok to release a version while keeping all of them here ..

Copy link
Contributor

@ohadbitt ohadbitt left a comment

Choose a reason for hiding this comment

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

some last nits

docs/KustoSink.md Outdated Show resolved Hide resolved
docs/KustoSink.md Outdated Show resolved Hide resolved
Co-authored-by: ohad bitton <[email protected]>
Co-authored-by: ohad bitton <[email protected]>
@ag-ramachandran ag-ramachandran merged commit fff00e0 into master Jan 2, 2025
4 checks passed
@ag-ramachandran ag-ramachandran deleted the feature/AddKustoStreamingFeature branch January 2, 2025 10:02
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.

4 participants