Skip to content

Conversation

@ruanwenjun
Copy link
Member

Purpose of the pull request

close #17467

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)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@ruanwenjun ruanwenjun requested a review from SbloodyS as a code owner September 3, 2025 03:49
@ruanwenjun ruanwenjun marked this pull request as draft September 3, 2025 03:50
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch 7 times, most recently from a6e1e5c to 525f81d Compare September 3, 2025 12:00
@ruanwenjun ruanwenjun changed the title [Improvement-17467][Master] Using a seperate datasource for quartz [Improvement-17467][Master] Support set a separate dataSource for quartz Sep 3, 2025
@ruanwenjun ruanwenjun added improvement make more easy to user or prompt friendly priority:middle labels Sep 3, 2025
@ruanwenjun ruanwenjun added this to the 3.3.2 milestone Sep 3, 2025
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch 2 times, most recently from d6d9488 to 663891f Compare September 3, 2025 12:01
@ruanwenjun ruanwenjun marked this pull request as ready for review September 3, 2025 12:02
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch from 663891f to a5503b8 Compare September 4, 2025 01:49
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch from a5503b8 to 8e71e45 Compare September 4, 2025 06:15
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch from 8e71e45 to f33d2a6 Compare September 4, 2025 09:19
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch 2 times, most recently from bbddb51 to ee25dc0 Compare September 8, 2025 11:44
@SbloodyS SbloodyS requested a review from zhongjiajie September 9, 2025 01:30
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch 3 times, most recently from 0984029 to b58d34d Compare September 15, 2025 13:28
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_quartzUseSeperateDataSource branch from b58d34d to f6d0f98 Compare September 15, 2025 15:01
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 60%)

See analysis details on SonarQube Cloud

@SbloodyS SbloodyS merged commit e3b97e6 into apache:dev Sep 16, 2025
70 of 71 checks passed
davidzollo pushed a commit to davidzollo/dolphinscheduler that referenced this pull request Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement][Master] Using a seperate datasource for quartz

4 participants