Skip to content

Improve cleanAccount stored procedure to accept a list of accounts#4

Open
sweep-ai-deprecated[bot] wants to merge 4 commits into
masterfrom
sweep/improve-cleanAccount-stored-procedure
Open

Improve cleanAccount stored procedure to accept a list of accounts#4
sweep-ai-deprecated[bot] wants to merge 4 commits into
masterfrom
sweep/improve-cleanAccount-stored-procedure

Conversation

@sweep-ai-deprecated

Copy link
Copy Markdown

Description

This PR modifies the cleanAccount stored procedure to accept a list of account IDs instead of a single account ID. This allows the procedure to clean multiple accounts in a single call, improving efficiency and usability. The changes are made to the following files:

  • util/src/main/resources/cleanAccount.sql
  • util/src/main/resources/cleanAccount-postgresql.sql
  • util/src/main/resources/trimAccount.sql
  • util/src/main/resources/trimAccount-postgresql.sql

Summary of Changes

  • Modified the cleanAccount stored procedure to accept an array of account IDs as a parameter.
  • Iterated over the array of account IDs to perform the deletion operation for each account ID.
  • Updated the cleanAccount-postgresql stored procedure to match the changes made in cleanAccount.
  • Modified the trimAccount stored procedure to accept an array of account IDs as a parameter.
  • Iterated over the array of account IDs to perform the deletion operation for each account ID.
  • Updated the trimAccount-postgresql stored procedure to match the changes made in trimAccount.

Fixes #2.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/improve-cleanAccount-stored-procedure

To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep Assigns Sweep to an issue or pull request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sweep: improve cleanAccount stored procedure to accept a list of accounts

0 participants