fix json map conversion in http proxy #8508

siarheivesialou · 2024-08-30T06:30:14Z

No description provided.

github-actions · 2024-08-30T06:33:30Z

⚪ 2024-08-30 06:33:30 UTC Pre-commit check linux-x86_64-release-clang14 for 2d24302 has started.
⚪ 2024-08-30 06:36:14 UTC ya make is running...
🟢 2024-08-30 06:42:32 UTC Build successful.

github-actions · 2024-08-30T06:33:54Z

⚪ 2024-08-30 06:33:53 UTC Pre-commit check linux-x86_64-relwithdebinfo for 2d24302 has started.
⚪ 2024-08-30 06:36:41 UTC ya make is running...
⚫ 2024-08-30 06:48:48 UTC Check cancelled

github-actions · 2024-08-30T06:33:56Z

⚪ 2024-08-30 06:33:55 UTC Pre-commit check linux-x86_64-release-asan for 2d24302 has started.
⚪ 2024-08-30 06:36:40 UTC ya make is running...
⚫ 2024-08-30 06:48:51 UTC Check cancelled

…_proxy and fix returning MD5 of message attributes in SendMessageBatch

github-actions · 2024-08-30T06:51:28Z

⚪ 2024-08-30 06:51:27 UTC Pre-commit check linux-x86_64-release-clang14 for fd41891 has started.
⚪ 2024-08-30 06:54:11 UTC ya make is running...
🟢 2024-08-30 07:00:25 UTC Build successful.

github-actions · 2024-08-30T06:51:50Z

⚪ 2024-08-30 06:51:50 UTC Pre-commit check linux-x86_64-release-asan for fd41891 has started.
⚪ 2024-08-30 06:54:31 UTC ya make is running...
🔴 2024-08-30 08:20:58 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
9840	9783	0	14	22	21

🟢 2024-08-30 08:21:50 UTC Build successful.
🟡 2024-08-30 08:22:20 UTC ydbd size 5.5 GiB changed* by +100.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash	main: `ea7bd23`	merge: `fd41891`	diff	diff %
ydbd size	5 945 024 016 Bytes	5 945 127 016 Bytes	+100.6 KiB	+0.002%
ydbd stripped size	1 492 882 640 Bytes	1 492 934 800 Bytes	+50.9 KiB	+0.003%

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

ydb/core/http_proxy/json_proto_conversion.h

github-actions · 2024-08-30T06:53:27Z

⚪ 2024-08-30 06:53:27 UTC Pre-commit check linux-x86_64-relwithdebinfo for fd41891 has started.
⚪ 2024-08-30 06:56:09 UTC ya make is running...
🟡 2024-08-30 07:54:14 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
14285	12903	0	3	1355	24

⚪ 2024-08-30 07:55:28 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-08-30 08:04:10 UTC Tests successful.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
29 (only retried tests)	16	0	0	0	13

🟢 2024-08-30 08:04:17 UTC Build successful.
🟡 2024-08-30 08:04:56 UTC ydbd size 8.2 GiB changed* by +106.1 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash	main: `a261c7b`	merge: `fd41891`	diff	diff %
ydbd size	8 820 643 840 Bytes	8 820 752 448 Bytes	+106.1 KiB	+0.001%
ydbd stripped size	480 880 712 Bytes	480 888 712 Bytes	+7.8 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}

ydb/core/http_proxy/json_proto_conversion.h

github-actions · 2024-08-30T12:20:32Z

⚪ 2024-08-30 12:20:32 UTC Pre-commit check linux-x86_64-release-asan for 4435559 has started.
⚪ 2024-08-30 12:23:08 UTC ya make is running...
🔴 2024-08-30 13:50:20 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
9833	9770	0	19	24	20

🟢 2024-08-30 13:51:15 UTC Build successful.
🟢 2024-08-30 13:51:47 UTC ydbd size 5.5 GiB changed* by +34.9 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash	main: `9fe138b`	merge: `4435559`	diff	diff %
ydbd size	5 944 988 984 Bytes	5 945 024 688 Bytes	+34.9 KiB	+0.001%
ydbd stripped size	1 492 870 544 Bytes	1 492 882 576 Bytes	+11.8 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}

github-actions · 2024-08-30T12:20:46Z

⚪ 2024-08-30 12:20:45 UTC Pre-commit check linux-x86_64-release-clang14 for 4435559 has started.
⚪ 2024-08-30 12:23:44 UTC ya make is running...
🟢 2024-08-30 12:30:24 UTC Build successful.

github-actions · 2024-08-30T12:20:48Z

⚪ 2024-08-30 12:20:48 UTC Pre-commit check linux-x86_64-relwithdebinfo for 4435559 has started.
⚪ 2024-08-30 12:23:22 UTC ya make is running...
🟡 2024-08-30 13:19:52 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
14279	12901	0	7	1354	17

⚪ 2024-08-30 13:21:18 UTC ya make is running... (failed tests rerun, try 2)
🟡 2024-08-30 13:28:15 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
29 (only retried tests)	15	0	1	0	13

⚪ 2024-08-30 13:28:23 UTC ya make is running... (failed tests rerun, try 3)
🟢 2024-08-30 13:35:24 UTC Tests successful.

Test history | Ya make output

TESTS	PASSED	ERRORS	FAILED	SKIPPED	MUTED^?
19 (only retried tests)	6	0	0	0	13

🟢 2024-08-30 13:35:32 UTC Build successful.
🟡 2024-08-30 13:36:11 UTC ydbd size 8.2 GiB changed* by +115.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash	main: `9fe138b`	merge: `4435559`	diff	diff %
ydbd size	8 820 202 192 Bytes	8 820 320 120 Bytes	+115.2 KiB	+0.001%
ydbd stripped size	480 882 120 Bytes	480 886 984 Bytes	+4.8 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}

github-actions bot added the new-feature label Aug 30, 2024

siarheivesialou requested a review from Alek5andr-Kotov August 30, 2024 06:30

github-actions bot removed the new-feature label Aug 30, 2024

siarheivesialou force-pushed the fix_json_map_conversion_in_http_proxy branch from 6477839 to 419fb5c Compare August 30, 2024 06:48

LOGBROKER-8891: fix inability to parse json into protobuf map in http…

8a9234d

…_proxy and fix returning MD5 of message attributes in SendMessageBatch

siarheivesialou force-pushed the fix_json_map_conversion_in_http_proxy branch from 419fb5c to 8a9234d Compare August 30, 2024 06:50

siarheivesialou commented Aug 30, 2024

View reviewed changes

ydb/core/http_proxy/json_proto_conversion.h Show resolved Hide resolved

Alek5andr-Kotov reviewed Aug 30, 2024

View reviewed changes

ydb/core/http_proxy/json_proto_conversion.h Show resolved Hide resolved

Alek5andr-Kotov reviewed Aug 30, 2024

View reviewed changes

ydb/core/http_proxy/json_proto_conversion.h Outdated Show resolved Hide resolved

fix map conversion depth

d0885ab

Alek5andr-Kotov approved these changes Sep 2, 2024

View reviewed changes

siarheivesialou merged commit fd6e5db into ydb-platform:main Sep 2, 2024
10 of 12 checks passed

siarheivesialou added a commit to siarheivesialou/ydb that referenced this pull request Sep 2, 2024

fix json map conversion in http proxy (ydb-platform#8508)

453fd76

siarheivesialou mentioned this pull request Sep 4, 2024

fix memory leak in ymq proxy #8706

Merged

This was referenced Sep 4, 2024

YQ-3151 added issues for timeout and cancelled #8718

Merged

YQ kqprun added cancel after for script requests #8768

Merged

Fixed unit test ManyColumnShards #8777

Merged

This was referenced Sep 9, 2024

Fix CLI max partitions default #8944

Merged

Fix reading without consumer from federation #9136

Merged

StekPerepolnen mentioned this pull request Sep 17, 2024

doc hc api images source #9376

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix json map conversion in http proxy #8508

fix json map conversion in http proxy #8508

siarheivesialou commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

fix json map conversion in http proxy #8508

fix json map conversion in http proxy #8508

Conversation

siarheivesialou commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

github-actions bot commented Aug 30, 2024 • edited Loading

siarheivesialou commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading

github-actions bot commented Aug 30, 2024 •

edited

Loading