chore(expr): cleanup all RwErrors

[BugenZhao]

Signed-off-by: Bugen Zhao [email protected]

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.

What's changed and what's your intention?

As title.

Checklist

• I have written necessary rustdoc comments
• I have added necessary unit tests and integration tests
• All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

• Tracking: Eliminate usages of RwError #4077

[BugenZhao]

May let #4862 merge first.

[stdrc]

LGTM

[codecov]

Codecov Report

Merging #4873 (87908c8) into main (aa6aebd) will increase coverage by 0.13%.
The diff coverage is 80.51%.

@@            Coverage Diff             @@
##             main    #4873      +/-   ##
==========================================
+ Coverage   73.80%   73.93%   +0.13%     
==========================================
  Files         871      873       +2     
  Lines      131635   132637    +1002     
==========================================
+ Hits        97148    98066     +918     
- Misses      34487    34571      +84     

Flag	Coverage Δ
rust	73.93% <80.51%> (+0.13%)	⬆️

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

Impacted Files	Coverage Δ
src/batch/src/executor/mod.rs	72.30% <ø> (ø)
src/batch/src/rpc/service/task_service.rs	0.00% <0.00%> (ø)
src/cmd_all/src/playground.rs	0.00% <0.00%> (ø)
src/common/src/util/mod.rs	0.00% <ø> (ø)
src/compute/src/rpc/service/stream_service.rs	0.00% <0.00%> (ø)
src/compute/src/server.rs	0.00% <0.00%> (ø)
src/connector/src/source/datagen/source/reader.rs	85.30% <ø> (-0.41%)	⬇️
src/expr/src/error.rs	25.00% <0.00%> (-15.00%)	⬇️
src/expr/src/expr/agg.rs	50.00% <0.00%> (ø)
src/expr/src/expr/expr_is_null.rs	90.21% <ø> (ø)
... and 98 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more