[DO NOT MERGE] Direct TCP | Add support for request timelines to be used in diagnostic strings #7165
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Convert credential scopes param to TokenRequest * Fix build errors in event hubs * Change .block() to StepVerifier * Fix tests * checkstyle .* imports
* Rename clients + annotations * add get/set to model types * 4.0.0-preview.4 * Fixing issues * Implementing PR comments Moving microsoft-azure to azure * updating pom.service to reflect new module names * changing parent to pom.client.xml * Removed recursive calls
* CI-fixes * Updated query metrics validation checks * Fixed stored procedure path in test * Fixed emulator path to public emulator
* Porting reactor netty changes from v3 to v4 Used Reactor-Core and Reactor-Netty latest release. New TCP Configurations with idle connection timeout, connect timeout, connection keep alive, etc. Used Reactor-Netty's response APIs to convert Flux / Mono to Flux / Mono * Porting NPE from V3 Port of: #5804 * Changing the max connection pool size to 1000 * Port from : #5862
* Changing packages from com.azure.data.cosmos -> com.azure.cosmos * updating the cosmos benchmark readme path after package name change * updating the cosmos readme path after package name change * Setting the correct version for v4 preview release: 4.0.0-preview.1
* Fixed constructors of sync responses by setting resource response in parent CosmosResponse class * Added tests for invoking responseWrapper for sync items
* Upgrading azure core version to 1.0.0 Fixing imports
* Updated changelog and readme * Minor update
* Diagnostic improvements based on recent changes to 2.6.X * Tweaked diagnostic messages * Tweaks for diagnostics * Tweaks for cloud test runs * Tweaks for performance test run * Tidied two files and tweaked ReadMyWriteWorkflow for performace test run on cloud * Tidied some code and tweaked DEBUG messages for performance test run * Changes for verifying cross-partition query fix on azure cloud * Changes for verifying cross-partition query fix on azure cloud * Switched from DEBUG to INFO-level logging * Tweaked benchmark logger settings * Added a newline
* adding partition migration flag * minor code style fix * system.exit(.) will terminate the test suit, removed * minor code style, remove empty lines etc * true instead of boolean.tostirng
…ent has been updated while we are trying to write the current state like when check-pointing the continuation token - see https://github.com/Azure/azure-sdk-for-java/pull/6205/files (#6233)
* Updated package name from internal to implementation * Fixed typo from aure to azure
…t from Main.java, and attempted to address a memory leak in the RntbdRequestTimer
…ure-sdk-for-java into feature/cosmos/v4
…n long or int value should specify units
* Corrected package misspelling in log4j.properties, removed System.exit from Main.java, and attempted to address a memory leak in the RntbdRequestTimer * Fixed a merge error and resolved #5043, Variables that specify time in long or int value should specify units * Minor code cleanup in RntbdTransportClient * Fixed a second merge issue
…yer (#6312) * Updated collection cache layer logic with v2. Removed Flux.empty() and Mono.empty() and instead used ValueHolder<>(null) * Fixed query test issues * Fixed emulator breaking change * Fixed DocumentProducerTest * Reverted emulator change from this PR * checked out Collection Crud Test from feature/cosmos/v4 branch
* Corrected package misspelling in log4j.properties, removed System.exit from Main.java, and attempted to address a memory leak in the RntbdRequestTimer * Fixed a merge error and resolved #5043, Variables that specify time in long or int value should specify units * Minor code cleanup in RntbdTransportClient * Fixed a second merge issue * Re-enabled all tests * Attempt to fix a test break * Fixed a javadoc issue and bumped the value of RntbdRequestTimer.TIMER_RESOLUTION_IN_NANOS from 5 to 100 ms in the hopes of resolving a performance issue on Windows * Fixed a javadoc issues * Fixed a javadoc issue * Reenabled direct tcp backpressure tests * Fixed a javadoc issue * Updated collection cache layer logic with v2. Removed Flux.empty() and Mono.empty() and instead used ValueHolder<>(null) * Fixed query test issues * Fixed emulator breaking change * Attempt to fix CosmosPartitionKeyTests::testNonPartitionedCollectionOperations test failure which resulted from an incorrectly formed expected result string. Also: Removed unnecessary Ignore annotation imports. * Fixed DocumentProducerTest * Tweaked code comments * Renamed beforeMethods for easier interpretation of the TestNG logs. We might or might not keep the renames. * Renamed beforeClass methods for easier interpretation of the TestNG logs. We might or might not keep the renames. * Workaround issue #6346 to identify test failures that might lie behind it. * Workaround issue #6346 to identify test failures that might lie behind it. * Attempt to resolve a number of test failures * Attempt to resolve a number of test failures * Attempt to resolve a number of test failures * Ignore ConsistencyTests1::validateConsistentPrefixOnAsyncReplication test because it is configured to use BoundedStaleness consistency. * Attempt to workaround VeryLargeDocumentQueryTest::queryLargeDocuments test timeout by bumping TIMEOUT as well as SETUP_TIMEOUT interval from 60 to 120 seconds * Attempt to workaround VeryLargeDocumentQueryTest::queryLargeDocuments test timeout on Emulator runs by moving the queryLargeDocuments test out of the emulator group into the simple group. * Updated test TODOs based on current set of issues * Added TODOs and bumped SETUP_TIMEOUT for those test classes that routinely timeout during intialization on compute/memory constrained systems like CI * Enabled LogLevelTest::createDocumentWithTraceLevelAtRoot * Added TODO and bumped timeout interval for before_OrderbyDocumentQueryTest which itermittently times out in CI environments. * Added TODO, bumped timeout interval, and revised error handling strategy for before_ParallelDocumentQueryTest which itermittently fails due to a 503 on a query operation in CI environments.
* async read use v4 api e2e * minor update * removed unnecessary BridgeInternalBenchmark.java * added a TOOD as per code review comment * remoed a leftover import
* DR fix in V4 * fixing build failure occure due to recent renaming of packages * removing flaky assert from the test
* Adding system information and gateway diagnostics * Adding more data for gateway * reverting redundant change * renaming variable name * resolving comments * creating OperatingSystemMXBean during class initialization * merging and resolving comment * Adding system information and gateway diagnostics * Adding more data for gateway * reverting redundant change * renaming variable name * resolving comments * creating OperatingSystemMXBean during class initialization * merging and resolving comment * resolving comments * Update CosmosResponseDiagnosticsTest.java Removing redundant system out
…k-for-java into issue/#6580/cosmos/v4/support-request-timeline
* improve Address Cache Physical address URI reconstruction * Physical addresses are constants, prior to this PR we used to parse physical address as string to URI on each request. This PR improves this. * replace String.format with concatination to avoid regex cost * SessionContainer ConcurrentHashMap lock contention improved * doing collectionResourceIdToSessionTokens.compute(collectionId, func) causes a contention for the active collectionId. This PR improves the contention lock * uri to url conversion (parsing) back and force eliminated * Uri to url conversaion parsing eliminated. * addressed review comments * fix broken tests * fix test
* benchmark added parallel in clause query, all query should measure latency * semaphore moded down
…k-for-java into issue/#6580/cosmos/v4/support-request-timeline
* checkstyle changes * fixing check style fixes * Renaming builders * Fixing checkstyle issues * Removing unwanted change * Fixing failing unit tests Removing throwing check exceptions from sync wrapper * Undoing incorrect text changes * Reverting the change that moves CosmosClientException to RuntimeException Moving back to Slf4j logger from azure core logger Moving back checkstyle fails to false. This has to enabled after deciding on the logger * - Removing excludes list from checkstyle. This can be added later when enabling checkstyle violations in the right place (eng/exceptions) - Implementing PR comments * Commit to retrigger tests
…k-for-java into issue/#6580/cosmos/v4/support-request-timeline
…tency and fixes earlier issues) (#6835) * use reactive stream concurreny for query * address code review comment
* Porting query plan from v3 to v4 Porting offset-limit from v3 to v4 Removing enableCrossPartitionQuery from FeedOptions * Fixing tests * Fixing tests * Merging v4 * Fixing tests * Increasing timeout on verylargedocument query test subscriber * Moving VeryLargeDocumentQueryTest to simple group from emulator as the test is passing on prod endpoint and failing on emulator when run locally Fixing id in a test * Increasing timeout for offerquery test as this is passing locally both on emulator and prod
* logging improvements * use v4 api in benchmark
…k-for-java into issue/#6580/cosmos/v4/support-request-timeline
…ure-sdk-for-java into issue/#6580/cosmos/v4/support-request-timeline
…evels into debug log-levels in prep for switching to the Azure Central SDK logger.
Author
|
/azp run java - cosmos - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
David-Noble-at-work
added
Cosmos
cosmos:v4-item
David-Noble-at-work
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
NOT READY FOR REVIEW
TO BE MERGED AFTER PR #7124.
See issue #6580.