Skip to content

Upgrade pgzip to v1.2.4#6189

Merged
deepthi merged 1 commit intovitessio:masterfrom
tinyspeck:am_upgrade_pgzip
May 15, 2020
Merged

Upgrade pgzip to v1.2.4#6189
deepthi merged 1 commit intovitessio:masterfrom
tinyspeck:am_upgrade_pgzip

Conversation

@ajm188
Copy link
Copy Markdown
Contributor

@ajm188 ajm188 commented May 15, 2020

This upgrades pgzip to include a fix to prevent negative byte counts being returned from pgzip.Write, which cause panics in bufio.

These panics manifest when using compressed xtrabackup backups as:

Error: <_Rendezvous of RPC that terminated with (StatusCode.UNKNOWN, caught panic during Backup: bufio: writer returned negative count from Write)>

Signed-off-by: Andrew Mason amason@slack-corp.com

This upgrades pgzip to include a
[fix](klauspost/pgzip#33) to prevent negative
byte counts being returned from `pgzip.Write`, which cause panics in
`bufio`.

These panics manifest when using compressed xtrabackup backups as:

```
Error: <_Rendezvous of RPC that terminated with
(StatusCode.UNKNOWN, caught panic during Backup: bufio: writer returned negative count from Write)>
```

Signed-off-by: Andrew Mason <amason@slack-corp.com>
@ajm188 ajm188 requested a review from sougou as a code owner May 15, 2020 17:22
@deepthi deepthi self-requested a review May 15, 2020 18:10
Copy link
Copy Markdown
Collaborator

@deepthi deepthi left a comment

Choose a reason for hiding this comment

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

LGTM

@deepthi deepthi merged commit b482cd0 into vitessio:master May 15, 2020
morgo added a commit to planetscale/vitess that referenced this pull request May 19, 2020
Backports vitessio#6189 to the releaes branch.

Or more specifically, the go.mod and go.sum were checked out from master
to release branch as there was also a change in gotest that should be
neutral.

Signed-off-by: Morgan Tocker <tocker@gmail.com>
morgo added a commit that referenced this pull request May 19, 2020
Backport upgrade pgzip #6189 to release branch
@deepthi deepthi added this to the v7.0 milestone Jul 27, 2020
@ajm188 ajm188 deleted the am_upgrade_pgzip branch January 14, 2021 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants