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

[cpp] Add Snappy library to Docker images for building C++ packages #8086

Merged
merged 1 commit into from
Sep 20, 2020

Conversation

massakam
Copy link
Contributor

Motivation

The program crashes when Snappy compression is enabled on the C++ client packaged as RPM/DEB. This is because Snappy library is not included in the Docker image for building the RPM/DEB package.
https://github.com/apache/pulsar/pull/4259/files#diff-bda8db832e446d092d38464123ed17bc
https://github.com/apache/pulsar/pull/4259/files#diff-7e441fcfcb20e339c71f7f8f48569579

Modifications

Added Snappy library to the docker images.

@massakam massakam added type/bug The PR fixed a bug or issue reported a bug component/c++ release/2.6.2 labels Sep 18, 2020
@massakam massakam added this to the 2.7.0 milestone Sep 18, 2020
@massakam massakam self-assigned this Sep 18, 2020
@jiazhai jiazhai merged commit f75289d into apache:master Sep 20, 2020
@massakam massakam deleted the snappy branch September 23, 2020 01:05
@massakam
Copy link
Contributor Author

Pushed these images to Docker Hub.

apachepulsar/pulsar-build:debian-9
apachepulsar/pulsar-build:centos-7

lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Oct 3, 2020
…#8086)

### Motivation

The program crashes when Snappy compression is enabled on the C++ client packaged as RPM/DEB. This is because Snappy library is not included in the Docker image for building the RPM/DEB package.
https://github.com/apache/pulsar/pull/4259/files#diff-bda8db832e446d092d38464123ed17bc
https://github.com/apache/pulsar/pull/4259/files#diff-7e441fcfcb20e339c71f7f8f48569579

### Modifications

Added Snappy library to the docker images.
wolfstudy pushed a commit that referenced this pull request Oct 30, 2020
### Motivation

The program crashes when Snappy compression is enabled on the C++ client packaged as RPM/DEB. This is because Snappy library is not included in the Docker image for building the RPM/DEB package.
https://github.com/apache/pulsar/pull/4259/files#diff-bda8db832e446d092d38464123ed17bc
https://github.com/apache/pulsar/pull/4259/files#diff-7e441fcfcb20e339c71f7f8f48569579

### Modifications

Added Snappy library to the docker images.

(cherry picked from commit f75289d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build release/2.6.2 type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants