* initial commit removing apache http libraries
* updated MSAL wrapper class to include headers in response
* Added TestHttpResponse class to replace BasicHttpResponse in Tests
* Added KustoParseException to replace ParseException from old Apache Client.
* removed todo regarding ParseException
* removed EofSensorInputStream
* added fixme to replace EofSensorInputStream
* Return of the Apache
* Fixed UtilitiesTest
Removed cast to Closeable in CloudInfo
* return UriBuilder to UriUtils
* add proxy options back to HttpClientFactory
* ran formatter
* closeable and fix build
* fix test
* format
* removed extra throws
* Per Ohad review comments:
-Adds custom HTTP status codes
-Integrates status codes
-Uses Context.NONE static factory
* removed caught exceptions that were rethrown with no changes/side effects in QueuedIngestClientImpl per review comments
* removes block on body by using BinaryData
removes Entity references
* ran formatter
* added Tracing object and builder to shorten method signatures
added HttpRequestBuilder to separate request building logic from execute methods in the client, providing better testability
Created BaseClient class to host the inner httpClient
Put post methods on BaseClient so no need for utils class
removed HttpPostUtils.java
updated impacted tests
left notes on Keep-Alive header in ClientFactory
ran formatter
* puts old URIBuilder back in place
* update readme with ProxyOptions
* aligns with PR 342
* fix issue with Closeable
* ran formatter
* Fix sync HttpWrapper implementation
* fix build
* format
* build
* Few bug fixes
Build works also with OKHttp providers
* readd the exclusion of commons-codec for security posture
* adds response timeout and client provider to the client properties wrapper
* removes references to Netty
* removes extra comment
* adds Keep-Alive back to prevent breaking change for users of Keep-Alive. updates CHANGELOG.md
ran formatter.
* removes deflate algorithm from encoding header
* revert removing Apache core (URIBuilder depends on utilities in core)
* remove netty from client factory
* fix
* removes CloseParentResourcesStream.java
* reusable Gzip logic
* factory now uses azure-core default http client options
* I always forget the formatter
* fix test
---------
Co-authored-by: cole <cole.snyder@gmork.tech>
Co-authored-by: Ohad Bitton <ohbitton@microsoft.com>
Co-authored-by: ohad bitton <32278684+ohadbitt@users.noreply.github.com>
* Added formatter plugin, right now only starting
* Put config on main file
* Updated style to match ours better.
* Added declaration.
* edit build.yml
* Update style and apply formatting
* Update style and apply formatting
* Tweaked settings again
* Replace with session because that might be the correct path.
* Try formatting
* line endings
* renormalize new lines
* csluuid
* new lines
* new lines
* Run again with clean
* fixed build.yml
* Changed style to be better
* Added some docs
* Initial commit.
The intent is to review this in its current feature-complete state, so I can also commit the equivalent changes in the Python SDK.
Then, in the next PR (or before merging this PR), I will make some additional improvements as discussed, including break main into individual methods and refactor this "query-ingest-query" approach into an additional "query-only" mode.
* Remove comment
* Optimize if/else
* Minor verbiage changes
* Minor verbiage changes
* Minor verbiage changes
* Update README.md
Co-authored-by: Yochai Gilad <yogilad@microsoft.com>
* - Break main up into methods
- Consider new parameters
- Few code review changes, like consistent periods at end of sentences and filename consistency with Python
* Only alter batching policy if !useExistingTable
* - Pass filesize of 0 for autodetection
- Do not create a File object to get its path; just pass along the provided path
- Move alterBatchingPolicy to within createNewTable, and wrap it in if(false) so it won't run
- Add comment explaining that we generally wouldn't print the output of a control command
* - Improve README text
- Change .alter command to .alter-merge command
* - Change filename kusto-sample-config.json to kusto_sample_config.json to be consistent with Python
* - Fix reference to Python
- Aad a comment for AppCertificate auth
Co-authored-by: Yochai Gilad <yogilad@microsoft.com>
asafmahlev
Cole
Yihezkel Schoenbrun
Ohad Bitton
ohad bitton
Eitan Moed
asaharn
Tanmaya Panda
ronmonetaMicro
Yihezkel Schoenbrun