github-actions[bot] opened a new pull request, #43129:
URL: https://github.com/apache/doris/pull/43129

   PR Body: ### What problem does this PR solve?
   Problem Summary:
   
   #### Issue 1
   There are some fields that is missing in audit log table.
   This PR add them all:
   - shuffle_send_rows
   - shuffle_send_bytes
   - scan_bytes_from_local_storage
   - scan_bytes_from_remote_storage
   - is_nereids
   - compute_group
   
   Notice that `compute_group` is previously name `cloudClusterName` in 
fe.audit.log,
   which is incorrect, so I change it to the right name.
   
   After this PR, all these fields will be saved in both audit log table and 
fe.audit.log
   
   #### Issue 2
   The `AuditEventBuilder` need to be reset at each run, the there will be 
duplicate audit log.
   
   #### Issue 3
   Add a new statement `call flush_audit_log()`. It will flush the audit log 
immediately to audit_log table.
   This is useful in test case, so that we don't need to wait 1min to flush the 
audit log data.
   
   ### Check List (For Committer)
   
   - Test <!-- At least one of them must be included. -->
   
       - [x] 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 colde files have been changed.
           - [ ] Other reason <!-- Add your reason?  -->
   
   - Behavior changed:
   
       - [ ] No.
       - [x] Yes. `cloudClusterName` in fe.audit.log change to `computeGroup`. 
And some fields are added to the audit log table.
   
   - Does this need documentation?
   
       - [ ] No.
       - [x] Yes. I will add this later
   
   - Release note
   
       <!-- bugfix, feat, behavior changed need a release note -->
       <!-- Add one line release note for this PR. -->
       [fix](auditlog) add missing audit log fields and duplicate audit log 
error
   
   ### 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 --> 
    Cherry-picked from #42262


-- 
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