0AyanamiRei opened a new pull request, #67353:
URL: https://github.com/apache/doris/pull/67353

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: None
   
   Problem Summary: `LOAD LABEL` submits an asynchronous Broker Load job but 
also generated a generic UUID `QUERY` profile without a planner, coordinator, 
or execution profile. That entry completed immediately with zero instances and 
obscured the real load profile. Profile summaries also copied the original LOAD 
SQL directly, exposing sensitive object-storage properties.
   
   This PR excludes `LoadCommand` from statement profiles and introduces a 
shared fail-closed SQL masker. It caches the masked SQL while the submit 
context and parsed `LoadCommand` are available, then uses it for the numeric 
Broker Load job profile without modifying the original statement needed for 
execution and replay.
   
   ### Release note
   
   Fix Broker Load profiles to remove empty submission query entries and mask 
sensitive SQL properties.
   
   ### Check List (For Author)
   
   - Test
       - [ ] Regression test
       - [x] Unit Test
           - `./run-fe-ut.sh --run 
org.apache.doris.qe.StmtExecutorTest,org.apache.doris.load.loadv2.BrokerLoadJobTest,org.apache.doris.nereids.parser.EncryptSQLTest`
 (47 tests passed)
       - [ ] Manual test (add detailed scripts or steps below)
       - [ ] No need to test or manual test. Explain why:
   - Behavior changed:
       - [ ] No.
       - [x] Yes. `LOAD LABEL` no longer emits an empty UUID `QUERY` profile, 
and Broker Load profile SQL masks sensitive properties.
   - Does this need documentation?
       - [x] No.
       - [ ] Yes.
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to