Skip to content

Fix test harness - update docker images#32

Merged
kantselovich merged 3 commits intomasterfrom
kantselovich-patch-2
Mar 6, 2026
Merged

Fix test harness - update docker images#32
kantselovich merged 3 commits intomasterfrom
kantselovich-patch-2

Conversation

@kantselovich
Copy link
Copy Markdown

@kantselovich kantselovich commented Jan 9, 2026

Fixes #33

Summary

  • fix the Presto Docker image build
  • remove Vertica from the default CI/local test stack
  • fix DuckDB schema introspection for catalog-qualified paths

Details

This PR takes the practical path to get CI green again.

Vertica is still supported in code, but the repo no longer provisions it by default in CI or docker-compose. Vertica-backed tests now run only when DATADIFF_VERTICA_URI is explicitly provided.

DuckDB also needed follow-up fixes:

  • render TIMESTAMP WITH TIME ZONE correctly
  • query information_schema.columns directly and filter by table_catalog for 3-part paths
  • update the corresponding unit test expectation

The Presto development image was also updated to use a current JDK base image and python-is-python3.

Testing

  • targeted DuckDB unit tests pass locally
  • CI running on the cleaned branch history

@kantselovich kantselovich changed the title Update Presto Dockerfile to use openjdk:11-jdk-slim Fix test harness - update docker images Jan 10, 2026
@kantselovich kantselovich marked this pull request as draft January 12, 2026 16:25
Copy link
Copy Markdown

@orca-security-us orca-security-us Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Failed Failed Vulnerabilities high 2   medium 0   low 0   info 0 View in Orca
☢️ The following Vulnerabilities (CVEs) have been detected
PACKAGE FILE CVE ID INSTALLED VERSION FIXED VERSION
critical deepdiff ./poetry.lock CVE-2025-58367 7.0.1 8.6.1 View in code
high cryptography ./poetry.lock CVE-2024-12797 43.0.3 44.0.1 View in code

Copy link
Copy Markdown

@orca-security-us orca-security-us Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@kantselovich kantselovich force-pushed the kantselovich-patch-2 branch from fe10855 to 84a3cba Compare March 6, 2026 02:18
@kantselovich kantselovich marked this pull request as ready for review March 6, 2026 02:22
@kantselovich kantselovich merged commit 4728a9c into master Mar 6, 2026
12 checks passed
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.

CI failures due to deprecated openjdk and vertica images

2 participants