[fix](test) replace hardcoded IP with variable in group commit redirect test#60573
[fix](test) replace hardcoded IP with variable in group commit redirect test#60573liaoxin01 wants to merge 1 commit intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
There was a problem hiding this comment.
Pull request overview
This PR fixes hardcoded IP addresses in the group commit redirect test by replacing them with the dynamic feIp variable. The test validates redirect behavior when enable_group_commit_streamload_be_forward is enabled, checking that the default (empty) redirect policy correctly routes to the FE's actual IP address rather than a hardcoded test IP.
Changes:
- Replaced hardcoded IP
11.20.20.19with$feIpvariable in default redirect policy assertions - Ensures test works correctly in dynamic/containerized environments where IPs are not fixed
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
regression-test/suites/load_p0/stream_load/test_group_commit_redirect.groovy
Show resolved
Hide resolved
regression-test/suites/load_p0/stream_load/test_group_commit_redirect.groovy
Show resolved
Hide resolved
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)