Skip to content

[Feature-12942][task-dataquality]The Data Quality Plugin supports the Resource Center#12947

Closed
lugela wants to merge 1 commit intoapache:devfrom
lugela:Feature-12942
Closed

[Feature-12942][task-dataquality]The Data Quality Plugin supports the Resource Center#12947
lugela wants to merge 1 commit intoapache:devfrom
lugela:Feature-12942

Conversation

@lugela
Copy link

@lugela lugela commented Nov 19, 2022

The resource center plays a great role when the Spark cluster using the data quality plug-in needs to use keytab file authentication or when submitting Spark tasks requires the use of third-party JAR.

Purpose of the pull request

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

@github-actions github-actions bot added backend document UI ui and front end related labels Nov 19, 2022
@lugela lugela changed the title [Feature-12942][task-dataquality] [Feature-12942][task-dataquality]The Data Quality Plugin supports the Resource Center Nov 19, 2022
…e Resource Center

The resource center plays a great role when the Spark cluster using the data quality plug-in needs to use keytab file authentication or when submitting Spark tasks requires the use of third-party JAR.
@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2022

Codecov Report

Merging #12947 (71d69db) into dev (2051868) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #12947      +/-   ##
============================================
- Coverage     39.36%   39.33%   -0.04%     
+ Complexity     4254     4251       -3     
============================================
  Files          1061     1061              
  Lines         40050    40053       +3     
  Branches       4604     4604              
============================================
- Hits          15765    15754      -11     
- Misses        22508    22526      +18     
+ Partials       1777     1773       -4     
Impacted Files Coverage Δ
.../parameters/dataquality/DataQualityParameters.java 0.00% <0.00%> (ø)
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 43.54% <0.00%> (-6.46%) ⬇️
...org/apache/dolphinscheduler/remote/utils/Host.java 42.55% <0.00%> (-2.13%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 51.38% <0.00%> (-1.39%) ⬇️

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

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@SbloodyS SbloodyS added 3.2.0 for 3.2.0 version feature new feature labels Nov 20, 2022
@SbloodyS SbloodyS added this to the 3.2.0 milestone Nov 20, 2022
@DarkAssassinator
Copy link
Contributor

currently there are many task plugin will running on bash, should we add a abstract bash parameter class, because now each task add these field is not a good idea. WDYT cc @caishunfeng @SbloodyS @ruanwenjun

@ruanwenjun
Copy link
Member

currently there are many task plugin will running on bash, should we add a abstract bash parameter class, because now each task add these field is not a good idea. WDYT cc @caishunfeng @SbloodyS @ruanwenjun

Which parameter should belong to bash? I don't think we need to do this.

@davidzollo davidzollo added the Waiting for user feedback Waiting for feedback from issue/PR author label Dec 3, 2022
@github-actions
Copy link

github-actions bot commented Apr 4, 2023

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Apr 4, 2023
@github-actions
Copy link

github-actions bot commented Jun 9, 2023

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.

@github-actions github-actions bot closed this Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.2.0 for 3.2.0 version backend feature new feature Stale UI ui and front end related Waiting for user feedback Waiting for feedback from issue/PR author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants