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.

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

Introduce locality counters for single node tx. (#13415) #13735

Open
wants to merge 1 commit into
base: stream-nb-24-3
Choose a base branch
from

Conversation

dcherednik
Copy link
Member

TotalSingleNodeReqCount - increases if tx touches the only one node (all shards located on the same node)
NonLocalSingleNodeReqCount - increases if tx touches the only one node and this node is same with session node (node where located kqp session and grpc request accepted)

  • New feature

Changelog entry

...

Changelog category

  • New feature
  • Experimental feature
  • Improvement
  • Performance improvement
  • Bugfix
  • Backward incompatible change
  • Documentation (changelog entry is not required)
  • Not for changelog (changelog entry is not required)

Additional information

...

TotalSingleNodeReqCount    -     increases if tx touches the only one node (all shards located on the same node)
NonLocalSingleNodeReqCount - increases if tx touches the only one node and this node is same with session node (node where located kqp session and grpc request accepted)

* New feature
@dcherednik dcherednik requested review from mvgorbunov, va-kuznecov and a team as code owners January 22, 2025 15:59
Copy link

github-actions bot commented Jan 22, 2025

2025-01-22 16:01:13 UTC Pre-commit check for f5e8f30 has started.
2025-01-22 16:04:08 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-01-22 17:14:31 UTC Build successful.
2025-01-22 17:14:57 UTC Tests are running...
🔴 2025-01-22 19:11:04 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
40423 35149 0 13 5217 44

🟢 2025-01-22 19:12:08 UTC ydbd size 8.3 GiB changed* by +94.1 KiB, which is < 100.0 KiB vs stream-nb-24-3: OK

ydbd size dash stream-nb-24-3: c0fc6a0 merge: f5e8f30 diff diff %
ydbd size 8 876 305 984 Bytes 8 876 402 368 Bytes +94.1 KiB +0.001%
ydbd stripped size 484 029 128 Bytes 484 031 528 Bytes +2.3 KiB +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Jan 22, 2025

2025-01-22 16:01:22 UTC Pre-commit check for f5e8f30 has started.
2025-01-22 16:04:21 UTC Build linux-x86_64-release-asan is running...
🟢 2025-01-22 16:54:11 UTC Build successful.
2025-01-22 16:54:28 UTC Tests are running...
🔴 2025-01-22 19:17:41 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11842 11716 0 34 45 47

🟢 2025-01-22 19:19:52 UTC ydbd size 5.6 GiB changed* by +74.9 KiB, which is < 100.0 KiB vs stream-nb-24-3: OK

ydbd size dash stream-nb-24-3: c0fc6a0 merge: f5e8f30 diff diff %
ydbd size 6 007 801 544 Bytes 6 007 878 248 Bytes +74.9 KiB +0.001%
ydbd stripped size 1 504 949 088 Bytes 1 504 969 504 Bytes +19.9 KiB +0.001%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@@ -1860,6 +1861,10 @@ class TKqpExecuterBase : public TActorBootstrapped<TDerived> {
LOG_N("Full stats: " << response.GetResult().GetStats());
}
}

for (const auto nodeId : ParticipantNodes) {
response.MutableResult()->AddParticipantNodes(nodeId);
Copy link
Member

Choose a reason for hiding this comment

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

это же локальное сообщение, зачем в протобуф перекладывать?

Copy link
Member Author

Choose a reason for hiding this comment

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

По аналогии со статистикой. Вообще там не очевидно как использовать поля в TEvTxResponse

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.

3 participants