Skip to content

fix: Register the s3:// scheme when an S3 connection is configured - #736

Open
sweb wants to merge 1 commit into
mainfrom
fix/register-s3-scheme
Open

fix: Register the s3:// scheme when an S3 connection is configured#736
sweb wants to merge 1 commit into
mainfrom
fix/register-s3-scheme

Conversation

@sweb

@sweb sweb commented Jul 28, 2026

Copy link
Copy Markdown
Member

Description

Right now, you need to tell Trino explicitly to write to s3a:// when issuing a CTAS against the Hive Metastore (e.g. for Iceberg tables) - by also registering the s3:// scheme, this is no longer required and can stay with the s3:// scheme.

Important to note: If we decide that this is a good direction, we should also add the s3:// scheme to the Spark configuration so that this works cross products without additional configuration.

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Author

  • Helm chart can be installed and deployed operator works
  • Integration tests passed (for non trivial changes)
  • Release note snippet added

Reviewer

  • Code contains useful comments
  • Code contains useful logging statements
  • (Integration-)Test cases added
  • Documentation added or updated. Follows the style guide.
  • Changelog updated

Acceptance

  • Feature Tracker has been updated
  • Proper release label has been added
  • Links to generated (nightly) docs added
  • Release note snippet added

@sweb
sweb force-pushed the fix/register-s3-scheme branch 2 times, most recently from 0f91b14 to 99e2b36 Compare July 28, 2026 13:20
@sweb
sweb marked this pull request as ready for review July 28, 2026 13:22
@sweb
sweb force-pushed the fix/register-s3-scheme branch from 99e2b36 to 4e9b53f Compare July 28, 2026 13:23
@sweb sweb moved this to Development: Waiting for Review in Stackable Engineering Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

1 participant