Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DSIP-87] Remove cache configuration of task #16892

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

ruanwenjun
Copy link
Member

Purpose of the pull request

close #16891

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 contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@github-actions github-actions bot added UI ui and front end related backend test e2e e2e test document labels Dec 11, 2024
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_removeCacheEnable branch from 23b4ee1 to 7ab60ba Compare December 12, 2024 01:44
@ruanwenjun ruanwenjun added this to the 3.3.0 milestone Dec 12, 2024
@ruanwenjun ruanwenjun force-pushed the dev_wenjun_removeCacheEnable branch 3 times, most recently from 9ddcf86 to cd5a3a9 Compare December 13, 2024 03:50
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_removeCacheEnable branch from 8d45d3a to 01b4edc Compare December 16, 2024 05:46
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@ruanwenjun ruanwenjun merged commit 1eedc16 into apache:dev Dec 16, 2024
69 of 70 checks passed
@ruanwenjun ruanwenjun deleted the dev_wenjun_removeCacheEnable branch December 16, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend document DSIP e2e e2e test test UI ui and front end related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DSIP-87] Remove cache configuration of task
3 participants