Skip to content

[fix](test) fix some test cases - #43217

Merged
morningman merged 4 commits into
apache:masterfrom
morningman:fix_43216
Nov 6, 2024
Merged

[fix](test) fix some test cases#43217
morningman merged 4 commits into
apache:masterfrom
morningman:fix_43216

Conversation

@morningman

@morningmanmorningman commented Nov 4, 2024

Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Problem Summary:

  1. add some missing output blocks for test_audit_log_behavior.out
  2. call flush_audit_log() does not support old planner, add check for it. followup [fix](auditlog) add missing audit log fields and duplicate audit log error #42262

Check List (For Committer)

  • Test

    • 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
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.
  • Release note

    None

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@doris-robot

Copy link
Copy Markdown

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@morningman

Copy link
Copy Markdown
ContributorAuthor

run buildall

@morningman

Copy link
Copy Markdown
ContributorAuthor

run buildall

@morningman

Copy link
Copy Markdown
ContributorAuthor

run buildall

@morningman

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Nov 6, 2024
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@morningman
morningman merged commit 395daa7 into apache:masterNov 6, 2024
morningman added a commit to morningman/doris that referenced this pull request Nov 6, 2024
### What problem does this PR solve?
Problem Summary:
1. add some missing output blocks for `test_audit_log_behavior.out`
2. `call flush_audit_log()` does not support old planner, add check for
it. followup apache#42262
morningman added a commit that referenced this pull request Nov 6, 2024
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
### What problem does this PR solve?
Problem Summary:
1. add some missing output blocks for `test_audit_log_behavior.out`
2. `call flush_audit_log()` does not support old planner, add check for
it. followup apache#42262
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.dev/2.1.7-mergeddev/3.0.3-mergedreviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@morningman@doris-robot@wuwenchi@zy-kkk@wm1581066