[fix][fn] Throw 404 RestException when state key not found

[jiangpengcheng]

Motivation

500 error is returned for querystate when the key doesn't exist.

it's related to #21597

Modifications

return 404 error when key doesn't exist

Verifying this change

• Make sure that the change passes the CI checks.

• Add tests to query non-exist state key in:

  • org.apache.pulsar.tests.integration.functions.PulsarStateTest.testSourceState
  • org.apache.pulsar.tests.integration.functions.PulsarStateTest.testSinkState

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

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository: jiangpengcheng#22

[Technoboy-]

Do we also need to revert the logic in the red range?

[Technoboy-]

Oh, can we add a test for this? We find this in the release stage.

[jiangpengcheng]

Oh, can we add a test for this? We find this in the release stage.

Sure, i will add a test tomorrow

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (22838ea) 36.46% compared to head (374ac87) 73.57%.

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #21921       +/-   ##
=============================================
+ Coverage     36.46%   73.57%   +37.11%     
- Complexity    12390    32390    +20000     
=============================================
  Files          1725     1861      +136     
  Lines        131701   138595     +6894     
  Branches      14401    15185      +784     
=============================================
+ Hits          48027   101978    +53951     
+ Misses        77254    28724    -48530     
- Partials       6420     7893     +1473     

Flag	Coverage Δ
inttests	24.12% <0.00%> (+0.01%)	⬆️
systests	23.68% <100.00%> (+0.01%)	⬆️
unittests	72.87% <0.00%> (+40.83%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ulsar/functions/worker/rest/api/ComponentImpl.java	47.92% <100.00%> (+26.37%)	⬆️

... and 1437 files with indirect coverage changes