You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudberry.apache.org by "ZTE-EBASE (via GitHub)" <gi...@apache.org> on 2026/09/11 07:36:21 UTC
[PR] Fix udp interconnect 2 [cloudberry]
ZTE-EBASE opened a new pull request, #1985:
URL: https://github.com/apache/cloudberry/pull/1985
What does this PR do?
Fix putIntoUnackQueueRing function logic for Intercontect interfaces
what
There are several logic issues in the putIntoUnackQueueRing function.
Defect : Misuse of TIMER_SPAN_LOSS (Severity: Medium)
why
For Defect (TIMER_SPAN_LOSS misuse):
Time alignment bias: Expected 5ms boundary alignment, actual 2.5ms alignment
Imprecise retransmission timing: May cause packets to trigger retransmission too early or too late
Degraded flow control performance: Affects accuracy of timeout-based mechanisms
Subtle bug: Not immediately obvious during testing but impacts long-term stability
Overall justification:
These are not just cosmetic issues—they affect correctness (Defect 2) and code quality (all three)
Low risk fix with high confidence: changes are localized and don't alter external behavior
Follows best practices: clean code, eliminate redundancy, use correct constants
How
Perform logical fixes on the premise that existing code functionality remains intact.
Type of Change
Bug fix (non-breaking change)
New feature (non-breaking change)
Breaking change (fix or feature with breaking changes)
Documentation update
Breaking Changes
Test Plan
Unit tests added/updated
Integration tests added/updated
Passed make installcheck
Passed make -C src/test installcheck-cbdb-parallel
Impact
Performance:
User-facing changes:
Dependencies:
Checklist
Followed [contribution guide](https://cloudberry.apache.org/contribute/code)
Added/updated documentation
Reviewed code for security implications
This PR contains AI-assisted code generation
Requested review from [cloudberry committers](https://github.com/orgs/apache/teams/cloudberry-committers)
Additional Context
CI Skip Instructions
--
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: commits-unsubscribe@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org
Re: [PR] Fix udp interconnect 2 [cloudberry]
Posted by "ZTE-EBASE (via GitHub)" <gi...@apache.org>.
ZTE-EBASE closed pull request #1985: Fix udp interconnect 2
URL: https://github.com/apache/cloudberry/pull/1985
--
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: commits-unsubscribe@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org