Skip to content

Change mem type from INT to BIGINT #936

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 21, 2021
Merged

Change mem type from INT to BIGINT #936

merged 2 commits into from
Mar 21, 2021

Conversation

splhack
Copy link
Contributor

@splhack splhack commented Mar 12, 2021

Same as #902, Cuebot can't handle frame complete report due to max RSS type INT.
Change all int_max_rss/vss type from INT to BIGINT.

com.imageworks.spcue.dispatcher.RqdRetryReportException: error processing the frame complete report, sending retry message to RQD org.springframework.dao.DataIntegrityViolationException: PreparedStatementCallback; SQL [UPDATE layer_mem SET int_max_rss = ? WHERE pk_layer = ? AND int_max_rss < ?]; ERROR: integer out of range; nested exception is org.postgresql.util.PSQLException: ERROR: integer out of range

@splhack splhack mentioned this pull request Mar 12, 2021
Merged
@bcipriano bcipriano merged commit 8371757 into AcademySoftwareFoundation:master Mar 21, 2021
@splhack splhack mentioned this pull request Mar 21, 2021
Merged
bcipriano pushed a commit that referenced this pull request Mar 23, 2021
@splhack
Fix #936 (#938)
d21ddcf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@larsbijl larsbijl larsbijl approved these changes

@bcipriano bcipriano bcipriano approved these changes

@DiegoTavares DiegoTavares Awaiting requested review from DiegoTavares DiegoTavares is a code owner

@gregdenton gregdenton Awaiting requested review from gregdenton

@IdrisMiles IdrisMiles Awaiting requested review from IdrisMiles

@jrray jrray Awaiting requested review from jrray

@smith1511 smith1511 Awaiting requested review from smith1511

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@splhack @larsbijl @bcipriano