Skip to content

Add myself to AUTHORS#41127

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
miretskiy:mod_authors
Sep 26, 2019
Merged

Add myself to AUTHORS#41127
craig[bot] merged 1 commit intocockroachdb:masterfrom
miretskiy:mod_authors

Conversation

@miretskiy
Copy link
Copy Markdown
Contributor

Authors += Myself

@miretskiy miretskiy requested a review from dt September 26, 2019 13:28
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Sep 26, 2019

CLA assistant check
All committers have signed the CLA.

@miretskiy miretskiy force-pushed the mod_authors branch 2 times, most recently from b25b5ef to 1d010fc Compare September 26, 2019 13:59
Release note: none
Release justification: no code changes
@miretskiy
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 26, 2019

Build failed

@miretskiy
Copy link
Copy Markdown
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request Sep 26, 2019
41098: roachtest/jepsen: don't fail test if retrieving invoke.log fails r=nvanbenschoten a=nvanbenschoten

Fixes #41062.

Release justification: Testing only.

Release note: None

41104: kv: check for 1-phase commit after request, not before r=nvanbenschoten a=nvanbenschoten

Found while verifying that #40518 didn't negatively impact YCSB performance.
As it turns out, most txns that restart hit the check in DistSender instead,
and that ends up being critical for forward progress. Without it, transactions
appear to starve because they never write intents.

The commit doesn't make any changes there, but it does fix an issue where
1PC transactions were being incorrectly detected. The detection was ignoring
the fact that a 1PC attempt could be rejected by DistSender or a Replica.
We now tie the metric to whether an EndTransaction actually evaluated as
a 1PC txn instead of tying it to whether the EndTransaction _wanted_ to
be evaluated as a 1PC txn.

Release justification: low risk and improves metric reporting

Release note: None

41127: Add myself to AUTHORS r=miretskiy a=miretskiy

Authors += Myself

Co-authored-by: Nathan VanBenschoten <nvanbenschoten@gmail.com>
Co-authored-by: Yevgeniy Miretskiy <yevgeniy@cockroachlabs.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 26, 2019

Build succeeded

@craig craig bot merged commit 2b56889 into cockroachdb:master Sep 26, 2019
@miretskiy miretskiy deleted the mod_authors branch September 26, 2019 17:46
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