flink icon indicating copy to clipboard operation
flink copied to clipboard

[FLINK-34512][rest] Thrown root cause for HandlerRequestException

Open masteryhx opened this issue 1 year ago • 1 comments

What is the purpose of the change

My Flink job only thrown below exception without the root cause:

org.apache.flink.runtime.rest.handler.HandlerRequestException: Cannot resolve path parameter (triggerid) from value "4c729cc8-ccc1-80dc-6c8d-9d84d52cfe7b".

It's better for HandlerRequest to thrown the root cause of HandlerRequestException.

Brief change log

  • Thrown the root cause of HandlerRequestException.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
  • The serializers: (yes / no / don't know)
  • The runtime per-record code paths (performance sensitive): (yes / no / don't know)
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
  • The S3 file system connector: (yes / no / don't know)

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

masteryhx avatar Feb 26 '24 07:02 masteryhx

CI report:

  • e3b6d323db9d2bb1bcd19a83d116a73cce18242a Azure: SUCCESS
Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

flinkbot avatar Feb 26 '24 07:02 flinkbot

Hi, Thanks all for reviewing. I found current description should be enough for me to find the root cause. Just close it. Thanks again!

masteryhx avatar Feb 28 '24 02:02 masteryhx