apache / seatunnel

SeaTunnel is a next-generation super high-performance, distributed, massive data integration tool.
https://seatunnel.apache.org/
Apache License 2.0
7.81k stars 1.76k forks source link

[Improvement] [Seatunnel-web] Improve code in DatasourceServiceImpl,SeaTunnelEngineMetricsExtractor etc. #7538

Closed arshadmohammad closed 2 weeks ago

arshadmohammad commented 2 weeks ago

Search before asking

What happened

While fixing bugs, had noted down few code improvement. JobExecParamUtil and JobUtils need to be renamed as per their usage and context. DatasourceServiceImpl is making extra DB call. SeaTunnelEngineMetricsExtractor, and DatasourceServiceImpl have some duplicate codes. Setting cluster name is redundant.

SeaTunnel Version

NA

SeaTunnel Config

NA

Running Command

NA

Error Exception

NA

Zeta or Flink or Spark Version

No response

Java or Scala Version

No response

Screenshots

No response

Are you willing to submit PR?

Code of Conduct

arshadmohammad commented 2 weeks ago

Raised PR https://github.com/apache/seatunnel-web/pull/199