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.

Sign up for GitHub

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

Jump to bottom

Implements azure core http client #388

Merged
merged 87 commits into from
Oct 27, 2024
Merged

Implements azure core http client #388

merged 87 commits into from
Oct 27, 2024

Conversation

The-Funk
Copy link
Contributor

No description provided.

cole and others added 30 commits March 25, 2024 20:29
Introduces KustoQuery wrapper object
b2b670d
@The-Funk
Merge branch 'master' into implement_kusto_query_object
ab1102c
@The-Funk
Merge branch 'master' into implement_kusto_query_object
77aaf02
working on async client
1f1f30d
updating client with KustoQuery object
939d4a7
updating client with KustoQuery object
64b100b
updating client with KustoQuery object
0989842
Merge remote-tracking branch 'origin/implement_kusto_query_object' in…
abd512d 
…to implement_kusto_query_object
@The-Funk
Merge branch 'master' into implement_kusto_query_object
7c07207
will simplify later.
4c4cc46
Merge remote-tracking branch 'origin/implement_kusto_query_object' in…
d000948 
…to implement_kusto_query_object
will simplify later.
8099b73
will simplify later.
610778e
@The-Funk
Merge branch 'master' into implement_kusto_query_object
5972c30
ran formatter, fixed test, fixed bad merge
39d479a
AtomicBooleans in already modified test. Save some lines, might as well.
c8c90a5
@The-Funk
Merge branch 'master' into implement_kusto_query_object
c59c124
hide KustoQuery object
a5b6d37
Merge remote-tracking branch 'origin/implement_kusto_query_object' in…
2ff7eda 
…to implement_kusto_query_object
rename and hide KustoRequest
e329eef
adding E2ETests for async queries
7e7ccc2
ran formatter
1c96d72
@The-Funk
no complete in sink
ab201e4
@The-Funk
move blocking operations off the io thread
96e3527
@The-Funk
run formatter
5cd3b31
@The-Funk
replace execute utilization
ae0c72e
fixes teardown. async still doesn't work yet.
2904187
fixes msal4j issue
4924c7d
@AsafMah
build.yaml
14fed03
@The-Funk
Introduces KustoQuery wrapper object
07df55e
@The-Funk The-Funk requested a review from ohadbitt October 8, 2024 23:38
ohadbitt
ohadbitt approved these changes Oct 15, 2024
View reviewed changes
Copy link
Collaborator

@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.

approving but wait for Asaf

AsafMah
AsafMah reviewed Oct 27, 2024
View reviewed changes
Copy link
Contributor

@AsafMah AsafMah left a comment

Choose a reason for hiding this comment

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

Approved, however I'll open a new issue to restore the timeout behavior.

@ohadbitt ohadbitt merged commit d73eeb3 into Azure:master Oct 27, 2024
2 of 6 checks passed
@yihezkel yihezkel mentioned this pull request Nov 10, 2024
Closed
@The-Funk The-Funk mentioned this pull request Nov 17, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ohadbitt ohadbitt ohadbitt approved these changes

@AsafMah AsafMah AsafMah approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@The-Funk @AsafMah @ohadbitt