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

Direct 3.21.0: Adds new API changes #2687

Merged

j82w merged 18 commits into master from users/jawilley/direct/3.21.0

Sep 2, 2021

Contributor

j82w commented Aug 26, 2021

Pull Request Template

Description

Add new system usage telemetry
Adds new ClientSideRequestStats interface to fix the start and end time
Fixes lock on client telemetry logging
Adds optimization to DiagnosticsHandler
Removes duplicate CPU collector in HA layer

Type of change

Please delete options that are not relevant.

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Closing issues

To automatically close an issue: closes #IssueNumber


          Bump SDK to 3.21.0

97ea71f

j82w requested review from ealsur, FabianMeiswinkel, khdang, kirankumarkolli, kirillg, neildsh and sboshra as code owners

August 26, 2021 21:23

ealsur reviewed

View reviewed changes

Microsoft.Azure.Cosmos/src/Tracing/TraceData/ClientSideRequestStatisticsTraceDatum.cs Outdated Show resolved Hide resolved


          Fix internal double blank line and update perf baseline

271a9eb

kirankumarkolli previously approved these changes

View reviewed changes


          DiagnosticsHandlerHelper fixes

de5557f

j82w dismissed kirankumarkolli’s stale review via

de5557f

August 27, 2021 13:19

Jake Willey and others added 2 commits

August 27, 2021 06:56


          Update query baseline

5ec898b


          Merge branch 'master' into users/jawilley/direct/3.21.0

43e3a85

j82w commented

View reviewed changes

...tests/Microsoft.Azure.Cosmos.Tests/BaselineTest/TestBaseline/QueryPlanBaselineTests.Like.xml Show resolved Hide resolved

kirankumarkolli reviewed

View reviewed changes

Microsoft.Azure.Cosmos/src/Handler/DiagnosticsHandler.cs Show resolved Hide resolved

kirankumarkolli reviewed

View reviewed changes

Microsoft.Azure.Cosmos/src/Handler/DiagnosticsHandlerHelper.cs Show resolved Hide resolved

kirankumarkolli reviewed

View reviewed changes

Microsoft.Azure.Cosmos/src/Handler/DiagnosticsHandlerHelper.cs Show resolved Hide resolved

kirankumarkolli reviewed

View reviewed changes

Microsoft.Azure.Cosmos/src/Handler/DiagnosticsHandlerHelper.cs Show resolved Hide resolved

kirankumarkolli reviewed

View reviewed changes

....Azure.Cosmos/tests/Microsoft.Azure.Cosmos.Performance.Tests/Contracts/BenchmarkResults.json Outdated Show resolved Hide resolved

Jake Willey and others added 9 commits

August 27, 2021 12:10


          Fix baseline tests

a89a29f


          Merge branch 'users/jawilley/direct/3.21.0' of https://github.com/Azu…

0ba7104

…re/azure-cosmos-dotnet-v3 into users/jawilley/direct/3.21.0


          Fix tests

1b70afa


          Merge branch 'master' into users/jawilley/direct/3.21.0

8b61357


          Compute added fields are not used in the OSS clone removing them from…

… github repo.


          Merge branch 'users/jawilley/direct/3.21.0' of https://github.com/Azu…

6dbc594

…re/azure-cosmos-dotnet-v3 into users/jawilley/direct/3.21.0


          Update baseline

0f82b28


          Fix internal builds

604c0d4


          Fix internal build

d695ea7

Jake Willey added 4 commits

August 31, 2021 11:55


          Update regions

a87e638


          Fix test


          Fix diagnostic handler helper name

af4335c


          Update perf bench after fixes

b44fa68

kirankumarkolli approved these changes

View reviewed changes

j82w merged commit 43f7887 into master

j82w deleted the users/jawilley/direct/3.21.0 branch

September 2, 2021 16:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

ealsur ealsur left review comments

kirankumarkolli kirankumarkolli approved these changes

FabianMeiswinkel Awaiting requested review from FabianMeiswinkel FabianMeiswinkel is a code owner

khdang Awaiting requested review from khdang khdang is a code owner

kirillg Awaiting requested review from kirillg kirillg is a code owner

neildsh Awaiting requested review from neildsh neildsh is a code owner

sboshra Awaiting requested review from sboshra sboshra is a code owner

Labels

None yet