Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2024-03-15 Thread via GitHub
doris-robot commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1999515236 TeamCity be ut coverage result: Function Coverage: 34.94% (8573/24533) Line Coverage: 26.65% (69480/260761) Region Coverage: 25.94% (36096/139174) Branch Coverage:

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2024-03-15 Thread via GitHub
github-actions[bot] commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1999373427 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2024-03-15 Thread via GitHub
github-actions[bot] commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1999368598 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2024-03-15 Thread via GitHub
github-actions[bot] commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1999368537 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2024-03-15 Thread via GitHub
morningman commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1999365585 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-11 Thread via GitHub
xiaokang commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806773980 #26179 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-11 Thread via GitHub
xiaokang merged PR #26805: URL: https://github.com/apache/doris/pull/26805 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-11 Thread via GitHub
doris-robot commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806755797 TeamCity be ut coverage result: Function Coverage: 38.33% (7955/20755) Line Coverage: 30.20% (64599/213898) Region Coverage: 29.28% (33317/113797) Branch Coverage:

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
xiaokang commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806736410 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806707304 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806707294 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806697284 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26805: URL: https://github.com/apache/doris/pull/26805#issuecomment-1806686958 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-10 Thread via GitHub
ryanzryu opened a new pull request, #26805: URL: https://github.com/apache/doris/pull/26805 Thrift server reads the first four bytes of a packet as the packet size, and without setting the string_limit parameter, it will directly request the memory corresponding to the packet size

Re: [PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-02 Thread via GitHub
github-actions[bot] commented on PR #26329: URL: https://github.com/apache/doris/pull/26329#issuecomment-1790718612 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [fix](thrift)limit be and fe thrift server max pkg size,avoid accepting error or too large package causing OOM [doris]

2023-11-02 Thread via GitHub
ryanzryu opened a new pull request, #26329: URL: https://github.com/apache/doris/pull/26329 … ## Proposed changes Thrift server reads the first four bytes of a packet as the packet size, and without setting the string_limit parameter, it will directly request the memory