Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Removed the extra delete statement from removeTaskFromLimit #2611

Merged
merged 1 commit into from
Nov 30, 2021

Conversation

aravindanr
Copy link
Collaborator

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe):

Changes in this PR

The CassandraExecutionDAO.removeTaskFromLimit calls DELETE from task_def_limit ... twice. The second call is a wasteful no-op.

Alternatives considered

None

@github-actions
Copy link
Contributor

github-actions bot commented Nov 30, 2021

Unit Test Results

   161 files  ±0     161 suites  ±0   9m 8s ⏱️ +9s
1 130 tests ±0  1 130 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit a3df6df. ± Comparison against base commit d13dd37.

♻️ This comment has been updated with latest results.

@aravindanr aravindanr added the type: bug bugs/ bug fixes label Nov 30, 2021
@aravindanr aravindanr merged commit 9e27215 into main Nov 30, 2021
@aravindanr aravindanr deleted the remove_delete_stmt branch November 30, 2021 21:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug bugs/ bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants