Closed xczh closed 1 year ago
Thanks for looking into this! Have you taken a look at #193? I'd actually imagine this issue is solved by it (we're working to fix a CI bug, after which #193 will be merged).
Sorry I didn't notice #193. It seems that you are trying to remove the dependency of snappy
. However, the lack of snappy
dependency in the current trunk branch code will cause compilation failure. I think the default branch should always be compiled and run successfully. The user should be able to complete the test according to the instructions in the README document.
@xczh I completely agree. The bottleneck on the CI issue is far less than ideal, but we are actively working on it. There's nothing to stop us from merging #193 before the CI issue is fixed, but doing so sets a precedent I don't love.
I see. After the #193 is merged, if everything is OK, I think this issue can be closed too. Thank you for your kindly reply :wink:
Affected Branch
trunk
Basic Diagnostics
[X] I've pulled the latest changes on the affected branch and the issue is still present.
[X] The issue is reproducible in docker
Description
In order to reproduce the issue, follow these steps:
docker compose --file docker-compose-2pc.yml up --build
ordocker compose --file docker-compose-atomizer.yml up --build
.For twophase:
For atomizer:
It seems that it is caused by the lack of snappy compression library at runtime. I will try to fix this problem. A PR will be submitted later for processing :grinning:
Code of Conduct