Skip to content

Fix: missing COPY for third_party/parquet_format in Dockerfile.all#15

Closed
tsafin with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-failing-github-actions-job
Closed

Fix: missing COPY for third_party/parquet_format in Dockerfile.all#15
tsafin with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown

The "Build All-Formats Image" CI job was failing because parquet_types.h was unavailable at compile time inside the Docker builder stage.

Root Cause

CMakeLists.txt adds third_party/parquet_format as a private include directory for tpch_core (providing the Thrift-generated parquet_types.h consumed by parquet_footer_merge.cpp), but .docker/Dockerfile.all never copied that directory into the build context — every other third_party/ subdirectory was present except this one.

Change

Added the missing COPY instruction to the builder stage in .docker/Dockerfile.all:

COPY third_party/parquet_format third_party/parquet_format

Copilot AI changed the title [WIP] Fix failing GitHub Actions job "Build All-Formats Image" Fix: missing COPY for third_party/parquet_format in Dockerfile.all Jul 20, 2026
Copilot AI requested a review from tsafin July 20, 2026 16:03
@tsafin

tsafin commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Close by ef28145

@tsafin tsafin closed this Jul 21, 2026
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