[Improvement][Flink] Change Flink command into FLINK_HOME
Purpose of the pull request
fix: https://github.com/apache/dolphinscheduler/issues/16003
Brief change log
Verify this pull request
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md
Hi @pegasas, please fix UT.
Hi @pegasas, please fix UT.
Thanks, Done.
Please run mvn spotless:apply @pegasas
Please run
mvn spotless:apply@pegasas
Weired,
it seems my change has not been posted.
Thanks
LGTM, please fix the UT
Sorry I missed test in flink-stream.
Done and pass spotless check on my local machine.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 40.66%. Comparing base (
ec01df1) to head (b1f938d). Report is 1 commits behind head on dev.
:exclamation: Current head b1f938d differs from pull request most recent head 081f834
Please upload reports for the commit 081f834 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## dev #16004 +/- ##
============================================
- Coverage 40.67% 40.66% -0.01%
+ Complexity 5238 5237 -1
============================================
Files 1382 1382
Lines 46023 46023
Branches 4922 4922
============================================
- Hits 18719 18716 -3
- Misses 25379 25381 +2
- Partials 1925 1926 +1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code