Skip to content

[chore](cloud) process_unknown add warning log and improve msg#61695

Merged
gavinchou merged 4 commits intoapache:masterfrom
w41ter:dev/process-unknown-warning
Apr 1, 2026
Merged

[chore](cloud) process_unknown add warning log and improve msg#61695
gavinchou merged 4 commits intoapache:masterfrom
w41ter:dev/process-unknown-warning

Conversation

@w41ter
Copy link
Copy Markdown
Contributor

@w41ter w41ter commented Mar 25, 2026

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

$ curl 'localhost:5000/MetaService/http/unknown_api_xxxxx?token=greedisgood9999&instance_id=default_instance_id'
{
    "code": "OK",
    "msg": "no handler found for path: /MetaService/http/unknown_api_xxxxx"
}

Release note

None

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

@w41ter w41ter requested a review from gavinchou as a code owner March 25, 2026 03:07
@w41ter w41ter added the cloud label Mar 25, 2026
@w41ter w41ter requested a review from dataroaring as a code owner March 25, 2026 03:07
@hello-stephen
Copy link
Copy Markdown
Contributor

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

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@w41ter
Copy link
Copy Markdown
Contributor Author

w41ter commented Mar 25, 2026

run buildall

@w41ter
Copy link
Copy Markdown
Contributor Author

w41ter commented Mar 25, 2026

run buildall

@w41ter
Copy link
Copy Markdown
Contributor Author

w41ter commented Mar 25, 2026

run buildall

@doris-robot
Copy link
Copy Markdown

Cloud UT Coverage Report

Increment line coverage 100.00% (11/11) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 78.70% (1796/2282)
Line Coverage 64.42% (32286/50118)
Region Coverage 65.31% (16170/24759)
Branch Coverage 55.73% (8612/15454)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

PR approved by anyone and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Apr 1, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

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

@gavinchou gavinchou merged commit 0d269a4 into apache:master Apr 1, 2026
29 of 30 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 1, 2026
Issue Number: close #xxx

Related PR: #xxx

Problem Summary:
```
$ curl 'localhost:5000/MetaService/http/unknown_api_xxxxx?token=greedisgood9999&instance_id=default_instance_id'
{
    "code": "OK",
    "msg": "no handler found for path: /MetaService/http/unknown_api_xxxxx"
}
```

Release note: None
@w41ter w41ter deleted the dev/process-unknown-warning branch April 2, 2026 02:09
morningman pushed a commit that referenced this pull request Apr 2, 2026
Issue Number: close #xxx

Related PR: #xxx

Problem Summary:
```
$ curl 'localhost:5000/MetaService/http/unknown_api_xxxxx?token=greedisgood9999&instance_id=default_instance_id'
{
    "code": "OK",
    "msg": "no handler found for path: /MetaService/http/unknown_api_xxxxx"
}
```

Release note: None
yiguolei pushed a commit that referenced this pull request Apr 3, 2026
…e msg #61695 (#62011)

Cherry-picked from #61695

Co-authored-by: walter <maochuan@selectdb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. cloud dev/4.1.0-merged reviewed snapshot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants