w41ter opened a new pull request, #48761:
URL: https://github.com/apache/doris/pull/48761

   …tion during ingesting
   
   ### What problem does this PR solve?
   
   Issue Number: close #xxx
   
   Related PR: #xxx
   
   Problem Summary:
   
   In the previous PR #48467, a thread_local http connection is added to save 
the reconnect latency. However during the mixed workloads (both stream load and 
downloading binlog), the persistent connection will be blocked by the stream 
loading, which causes a latency spike.
   
   The ingest latency with a persistent connection:
   ![Ingest latency with thread_local persistent 
connection](https://github.com/user-attachments/assets/c47dfce7-8299-46a9-b683-581872e39564)
   
   The ingest latency without persistent connection:
   
   ![Ingest latency without thread_local persistent 
connection](https://github.com/user-attachments/assets/749cc356-00d7-4ba2-b6a4-185c85e76066)
   
   This PR makes a persistent connection for each ingesting task, and adding a 
config `enable_ingest_binlog_with_persistent_connection` to control whether to 
use the persistent connection. The config is disabled by default, and users who 
need to download binlogs over long distances can enable it to reduce the sync 
latency.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test <!-- At least one of them must be included. -->
       - [ ] Regression test
       - [ ] Unit Test
       - [ ] Manual test (add detailed scripts or steps below)
       - [ ] No need to test or manual test. Explain why:
           - [ ] This is a refactor/code format and no logic has been changed.
           - [ ] Previous test can cover this change.
           - [ ] No code files have been changed.
           - [ ] Other reason <!-- Add your reason?  -->
   
   - Behavior changed:
       - [ ] No.
       - [ ] Yes. <!-- Explain the behavior change -->
   
   - Does this need documentation?
       - [ ] No.
       - [ ] Yes. <!-- Add document PR link here. eg: 
https://github.com/apache/doris-website/pull/1214 -->
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label <!-- Add branch pick label that this PR should 
merge into -->
   
   


-- 
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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to