apache / datafusion-ballista

Apache DataFusion Ballista Distributed Query Engine
https://datafusion.apache.org/ballista
Apache License 2.0
1.39k stars 181 forks source link

docs: list protoc dependency #989

Closed davidwilemski closed 4 months ago

davidwilemski commented 4 months ago

Rationale for this change

While following the quick start guide, I ran into a compilation error when prost tried to build the proto files. Installing the protobuf compiler allowed me to follow the rest of the quick start guide.

I also noticed the manual setup section of the contributors guide had a similar list of dependencies that was missing the protobuf compiler.

What changes are included in this PR?

Since installing the protobuf compiler was sufficient for me to build the project, I thought it made sense to document this so others following the docs would be aware of the dependency.

Are there any user-facing changes?

No -- only documentation changes.