Skip to content

Conversation

@mayleaf
Copy link

@mayleaf mayleaf commented May 5, 2024

Solved #4555

  • Motivation
    On DELETE CASCADE on Foreign Keys #4555, We found delete methods in SimpleJobRepository are removing with separated SQL queries for each metadata tables
  • Modification
    Add ON DELETE CASCADE to each metadata tables' foreign keys.

@pivotal-cla
Copy link

@mayleaf Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@mayleaf Thank you for signing the Contributor License Agreement!

* @param jobExecutions a collection of {@link JobExecution}
*/
public void removeJobExecutions(Collection<JobExecution> jobExecutions) {
for (JobExecution jobExecution : jobExecutions) {
Copy link
Author

Choose a reason for hiding this comment

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

Can I delete this method since it's no longer used after this pull request?

@mayleaf mayleaf force-pushed the delete_cascade_on_foreign_keys branch from 61f5b39 to 092ba13 Compare October 19, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants