Skip to content

build(ci): Update the Presto Java image to 0.297#17226

Draft
czentgr wants to merge 2 commits intofacebookincubator:mainfrom
czentgr:cz_update_presto_0.297
Draft

build(ci): Update the Presto Java image to 0.297#17226
czentgr wants to merge 2 commits intofacebookincubator:mainfrom
czentgr:cz_update_presto_0.297

Conversation

@czentgr
Copy link
Copy Markdown
Collaborator

@czentgr czentgr commented Apr 17, 2026

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 17, 2026

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit cb019d4
🔍 Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/69e246bebca06a0008f70433

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 17, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 17, 2026

Build Impact Analysis

No build targets affected by this change.


Fast path • Graph from main@3da0390118fa4b0addbfbdeb03f438ec1b1237a6

@czentgr czentgr force-pushed the cz_update_presto_0.297 branch from 042ac4a to cb019d4 Compare April 17, 2026 14:42
@czentgr czentgr changed the title chore(ci): Update the Presto Java image to 0.297 build(ci): Update the Presto Java image to 0.297 Apr 17, 2026
@github-actions
Copy link
Copy Markdown

CI Failure Analysis

All 7 failed fuzzer jobs fail with the same root cause: the Presto Java server fails to start inside the new container image, causing every fuzzer that depends on Presto as a source of truth to abort.

🔴 Join Fuzzer — INFRA Failure View logs

🔴 RowNumber Fuzzer — INFRA Failure View logs

🔴 Expression Fuzzer with Presto SOT — INFRA Failure View logs

🔴 Window Fuzzer with Presto as source of truth — INFRA Failure View logs

🔴 TopNRowNumber Fuzzer — INFRA Failure View logs

🔴 Aggregation Fuzzer with Presto as source of truth — INFRA Failure View logs

🔴 Writer Fuzzer with Presto as source of truth — INFRA Failure View logs

Error (identical in all 7 jobs):

Error running command: java.net.ConnectException: Failed to connect to /127.0.0.1:8080

The Presto CLI (/opt/presto-cli --server 127.0.0.1:8080 --execute 'CREATE SCHEMA IF NOT EXISTS hive.tpch;') fails to connect after the 60-second startup wait. The Presto server ($PRESTO_HOME/bin/launcher run) never becomes reachable on port 8080. No fuzzer code ran — the jobs fail during Presto server initialization.


Correlation with PR changes:

This failure is directly caused by the PR changes. The PR makes two related changes:

  1. Container image switch (.github/workflows/scheduled.yml): All fuzzer jobs are changed from ghcr.io/facebookincubator/velox-dev:presto-javaghcr.io/czentgr/czentgr-test:presto-java
  2. Presto version bump (docker-compose.yml, scripts/docker/java.dockerfile): PRESTO_VERSION changed from 0.2950.297

The new container image ghcr.io/czentgr/czentgr-test:presto-java contains a Presto 0.297 server that fails to start. The uploaded server.log artifacts are only 205 bytes each, meaning the Presto server likely crashes immediately or never writes meaningful output.

Known issues:

  • No open issues track this specific failure.
  • The most recent Fuzzer Jobs run on main (run 24550311472) passed successfully, confirming this is not a pre-existing/flaky issue. The failure is specific to this PR's container image change.

Recommended fix:

  • Investigate why Presto 0.297 fails to start in the ghcr.io/czentgr/czentgr-test:presto-java container. Check the server logs for JVM/startup errors (the uploaded artifacts may contain clues).
  • Consider testing the container image locally first: run the container, start Presto, and verify it listens on port 8080 before pushing the CI change.
  • If Presto 0.297 requires additional JVM args, memory settings, or configuration changes, update the workflow or the container image accordingly.
  • If the image is still being built/tested, consider keeping the original ghcr.io/facebookincubator/velox-dev:presto-java container reference until the new image is validated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant