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

HPC-9551: Chunk model version's bulk update #250

Merged
merged 2 commits into from
May 7, 2024
Merged

HPC-9551: Chunk model version's bulk update #250

merged 2 commits into from
May 7, 2024

Conversation

enxtur
Copy link
Contributor

@enxtur enxtur commented May 6, 2024

When publishing custom version on a plan, a bulk update is triggered on versionModels, including attachmentVersion. However, when performing a bulk update with over 100k records which is more than batch limit, it throws a "bind message has 100K+ parameter formats but 0 parameters" error.
To address this, I'm implementing an update to versionModels in batches, as demonstrated practice in the following commit.
Additionally, the batch limit on my local environment is 2^16, and I assume the production environment has the same limit. However, the demo and blue dev environments have a batch limit of around 2^15. Therefore, I'm using the lower number here.

Copy link

@unocha-hpc unocha-hpc left a comment

Choose a reason for hiding this comment

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

Checks have passed and this pull request is ready for manual review

@enxtur enxtur marked this pull request as ready for review May 6, 2024 14:18
@enxtur enxtur requested a review from a team as a code owner May 6, 2024 14:18
@enxtur enxtur added ready for review All comments have been addressed, and the Pull Request is ready for review deploy to red.dev and removed needs testing deploy to blue.dev labels May 6, 2024
@Delgee Delgee assigned enxtur and unassigned Delgee May 7, 2024
@Delgee Delgee added needs minor changes There are review or issue comments to address and removed ready for review All comments have been addressed, and the Pull Request is ready for review labels May 7, 2024
Copy link

@unocha-hpc unocha-hpc left a comment

Choose a reason for hiding this comment

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

Checks have passed and this pull request is ready for manual review

@enxtur enxtur merged commit bd7a2bc into develop May 7, 2024
5 checks passed
@enxtur enxtur deleted the HPC-9551 branch May 7, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy to red.dev needs minor changes There are review or issue comments to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants