[Chore](runtime-filter) change some rf dcheck to exception - Githubissues

apache / doris

Apache Doris is an easy-to-use, high performance and unified analytics database.

https://doris.apache.org

Apache License 2.0

12.77k stars 3.29k forks source link

[Chore](runtime-filter) change some rf dcheck to exception #44354

Closed BiteTheDDDDt closed 1 day ago

BiteTheDDDDt commented 1 day ago

What problem does this PR solve?

change some rf dcheck to exception

Check List (For Author)

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 code files have been changed.
  - [ ] Other reason
Behavior changed:
- [ ] No.
- [ ] Yes.
Does this need documentation?
- [ ] No.
- [ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

BiteTheDDDDt commented 1 day ago

run buildall

github-actions[bot] commented 1 day ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 1 day ago

PR approved by anyone and no changes requested.

github-actions[bot] commented 1 day ago

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