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

[C++ SDK] Remove session from session pool if stream session was closed by server side. #13199

Merged
merged 6 commits into from
Jan 15, 2025

Conversation

dcherednik
Copy link
Member

Changelog category

  • Improvement

@dcherednik dcherednik requested a review from a team as a code owner January 8, 2025 16:50
Copy link

github-actions bot commented Jan 8, 2025

2025-01-08 16:54:01 UTC Pre-commit check linux-x86_64-relwithdebinfo for 07aaae9 has started.
2025-01-08 16:54:13 UTC Artifacts will be uploaded here
2025-01-08 16:57:08 UTC ya make is running...
🟡 2025-01-08 17:48:13 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
25848 23327 0 8 2387 126

2025-01-08 17:50:21 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-08 18:02:18 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
201 (only retried tests) 81 0 0 0 120

🟢 2025-01-08 18:02:25 UTC Build successful.
🟢 2025-01-08 18:02:45 UTC ydbd size 2.1 GiB changed* by +22.7 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9f31130 merge: 07aaae9 diff diff %
ydbd size 2 244 317 464 Bytes 2 244 340 664 Bytes +22.7 KiB +0.001%
ydbd stripped size 472 336 560 Bytes 472 337 712 Bytes +1.1 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 8, 2025

2025-01-08 16:54:21 UTC Pre-commit check linux-x86_64-release-asan for 07aaae9 has started.
2025-01-08 16:54:33 UTC Artifacts will be uploaded here
2025-01-08 16:57:36 UTC ya make is running...
🟡 2025-01-08 17:56:10 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11889 11826 0 31 2 30

2025-01-08 17:57:12 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-08 18:09:32 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
109 (only retried tests) 75 0 7 2 25

2025-01-08 18:09:41 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-01-08 18:21:16 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
63 (only retried tests) 30 0 6 0 27

🟢 2025-01-08 18:21:23 UTC Build successful.
🟢 2025-01-08 18:21:54 UTC ydbd size 3.6 GiB changed* by +56.9 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9f31130 merge: 07aaae9 diff diff %
ydbd size 3 873 834 368 Bytes 3 873 892 656 Bytes +56.9 KiB +0.002%
ydbd stripped size 1 350 684 240 Bytes 1 350 705 136 Bytes +20.4 KiB +0.002%

*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 10, 2025

2025-01-10 11:49:38 UTC Pre-commit check linux-x86_64-release-asan for 7139dfc has started.
2025-01-10 11:49:49 UTC Artifacts will be uploaded here
2025-01-10 11:52:41 UTC ya make is running...
🟡 2025-01-10 13:12:12 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11899 11847 0 16 2 34

2025-01-10 13:13:37 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-10 13:28:09 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
92 (only retried tests) 57 0 2 0 33

2025-01-10 13:28:18 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-01-10 13:41:26 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
65 (only retried tests) 36 0 2 1 26

🟢 2025-01-10 13:41:34 UTC Build successful.
🟢 2025-01-10 13:42:04 UTC ydbd size 3.6 GiB changed* by +39.8 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: b754b20 merge: 7139dfc diff diff %
ydbd size 3 874 904 408 Bytes 3 874 945 176 Bytes +39.8 KiB +0.001%
ydbd stripped size 1 350 991 344 Bytes 1 351 004 144 Bytes +12.5 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

Copy link

github-actions bot commented Jan 10, 2025

2025-01-10 11:49:47 UTC Pre-commit check linux-x86_64-relwithdebinfo for 7139dfc has started.
2025-01-10 11:49:58 UTC Artifacts will be uploaded here
2025-01-10 11:52:51 UTC ya make is running...
🟡 2025-01-10 13:07:47 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
25860 23348 0 5 2383 124

2025-01-10 13:09:54 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-10 13:23:33 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
192 (only retried tests) 73 0 0 0 119

🟢 2025-01-10 13:23:42 UTC Build successful.
🟢 2025-01-10 13:24:05 UTC ydbd size 2.1 GiB changed* by +14.2 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: b754b20 merge: 7139dfc diff diff %
ydbd size 2 244 935 256 Bytes 2 244 949 824 Bytes +14.2 KiB +0.001%
ydbd stripped size 472 443 792 Bytes 472 443 024 Bytes -768 Bytes -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 10, 2025

2025-01-10 16:49:11 UTC Pre-commit check linux-x86_64-release-asan for 34466f8 has started.
2025-01-10 16:49:16 UTC Artifacts will be uploaded here
2025-01-10 16:52:12 UTC ya make is running...
🟡 2025-01-10 17:56:50 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11900 11857 0 14 1 28

2025-01-10 17:58:30 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-10 18:10:38 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
75 (only retried tests) 45 0 2 2 26

2025-01-10 18:10:46 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-01-10 18:22:47 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
53 (only retried tests) 24 0 2 0 27

🟢 2025-01-10 18:22:55 UTC Build successful.
🟢 2025-01-10 18:23:23 UTC ydbd size 3.6 GiB changed* by +68.2 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: d425a70 merge: 34466f8 diff diff %
ydbd size 3 875 263 680 Bytes 3 875 333 480 Bytes +68.2 KiB +0.002%
ydbd stripped size 1 351 121 072 Bytes 1 351 144 496 Bytes +22.9 KiB +0.002%

*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 10, 2025

2025-01-10 16:49:48 UTC Pre-commit check linux-x86_64-relwithdebinfo for 34466f8 has started.
2025-01-10 16:50:00 UTC Artifacts will be uploaded here
2025-01-10 16:52:56 UTC ya make is running...
🟡 2025-01-10 17:43:53 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
25864 23359 0 1 2380 124

2025-01-10 17:46:12 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-10 17:59:15 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
182 (only retried tests) 63 0 0 0 119

🟢 2025-01-10 17:59:23 UTC Build successful.
🟢 2025-01-10 17:59:46 UTC ydbd size 2.1 GiB changed* by +26.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 77c1953 merge: 34466f8 diff diff %
ydbd size 2 245 128 320 Bytes 2 245 155 488 Bytes +26.5 KiB +0.001%
ydbd stripped size 472 483 120 Bytes 472 484 208 Bytes +1.1 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 13, 2025

2025-01-13 11:11:18 UTC Pre-commit check linux-x86_64-relwithdebinfo for 06036c3 has started.
2025-01-13 11:11:29 UTC Artifacts will be uploaded here
2025-01-13 11:14:28 UTC ya make is running...
🟡 2025-01-13 12:06:43 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
25869 23364 0 2 2380 123

2025-01-13 12:09:17 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-13 12:21:26 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
180 (only retried tests) 61 0 0 0 119

🟢 2025-01-13 12:21:32 UTC Build successful.
🟢 2025-01-13 12:21:51 UTC ydbd size 2.1 GiB changed* by +30.6 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: cbefb3a merge: 06036c3 diff diff %
ydbd size 2 245 767 800 Bytes 2 245 799 128 Bytes +30.6 KiB +0.001%
ydbd stripped size 472 632 976 Bytes 472 638 224 Bytes +5.1 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

Copy link

github-actions bot commented Jan 13, 2025

2025-01-13 11:11:18 UTC Pre-commit check linux-x86_64-release-asan for 06036c3 has started.
2025-01-13 11:11:29 UTC Artifacts will be uploaded here
2025-01-13 11:14:30 UTC ya make is running...
🟡 2025-01-13 12:11:12 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11903 11839 0 30 6 28

2025-01-13 12:12:16 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-13 12:24:33 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
99 (only retried tests) 67 0 1 4 27

2025-01-13 12:24:42 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-01-13 12:36:32 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
55 (only retried tests) 30 0 0 0 25

🟢 2025-01-13 12:36:39 UTC Build successful.
🟢 2025-01-13 12:37:07 UTC ydbd size 3.6 GiB changed* by +68.3 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: cbefb3a merge: 06036c3 diff diff %
ydbd size 3 893 386 600 Bytes 3 893 456 536 Bytes +68.3 KiB +0.002%
ydbd stripped size 1 360 209 168 Bytes 1 360 232 720 Bytes +23.0 KiB +0.002%

*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 15, 2025

2025-01-15 12:47:42 UTC Pre-commit check linux-x86_64-relwithdebinfo for 9646e1c has started.
2025-01-15 12:47:49 UTC Artifacts will be uploaded here
2025-01-15 12:51:15 UTC ya make is running...
🟡 2025-01-15 13:57:41 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
25882 23365 0 8 2381 128

2025-01-15 14:00:03 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-15 14:12:21 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
197 (only retried tests) 73 0 0 0 124

🟢 2025-01-15 14:12:28 UTC Build successful.
🟢 2025-01-15 14:12:51 UTC ydbd size 2.1 GiB changed* by +26.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 2555770 merge: 9646e1c diff diff %
ydbd size 2 247 501 384 Bytes 2 247 528 536 Bytes +26.5 KiB +0.001%
ydbd stripped size 472 866 896 Bytes 472 867 984 Bytes +1.1 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 15, 2025

2025-01-15 12:49:29 UTC Pre-commit check linux-x86_64-release-asan for 9646e1c has started.
2025-01-15 12:49:35 UTC Artifacts will be uploaded here
2025-01-15 12:52:31 UTC ya make is running...
🟡 2025-01-15 14:00:29 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11913 11868 0 11 8 26

2025-01-15 14:01:59 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-01-15 14:13:49 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
73 (only retried tests) 43 0 0 5 25

🟢 2025-01-15 14:13:56 UTC Build successful.
🟢 2025-01-15 14:14:20 UTC ydbd size 3.6 GiB changed* by +71.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 1edb570 merge: 9646e1c diff diff %
ydbd size 3 895 435 096 Bytes 3 895 508 336 Bytes +71.5 KiB +0.002%
ydbd stripped size 1 360 977 648 Bytes 1 360 999 344 Bytes +21.2 KiB +0.002%

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

@dcherednik dcherednik merged commit f322990 into ydb-platform:main Jan 15, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants