Skip to content

Don't run testTrinoPreservesReaderAndWriterVersions in Databricks 7.3#14687

Merged
ebyhr merged 1 commit intomasterfrom
ebi/delta-fix-test
Oct 21, 2022
Merged

Don't run testTrinoPreservesReaderAndWriterVersions in Databricks 7.3#14687
ebyhr merged 1 commit intomasterfrom
ebi/delta-fix-test

Conversation

@ebyhr
Copy link
Copy Markdown
Member

@ebyhr ebyhr commented Oct 19, 2022

Description

Don't run testTrinoPreservesReaderAndWriterVersions in Databricks 7.3
CI was green because suite-delta-lake-databricks didn't run since #10984 (#14662 fixes the issue)

Non-technical explanation

Release notes

(x) This is not user-visible or docs only and no release notes are required.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why the exclusion?

Copy link
Copy Markdown
Member Author

@ebyhr ebyhr Oct 19, 2022

Choose a reason for hiding this comment

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

Version 7.3 doesn't support delta.minReaderVersion and delta.minWriterVersion table properties. It causes query failure.

I will take a look if we shouldn't write those properties when existing tables don't have those properties.

io.trino.tempto.query.QueryExecutionException: java.sql.SQLException: [Databricks][DatabricksJDBCDriver](500051) ERROR processing query/statement. Error Code: 0, SQL state: Error running query: org.apache.spark.sql.AnalysisException: Unknown configuration was specified: delta.minWriterVersion;, Query: CREATE TABLE default.test_trino_preserves_versions_vtizu3eknquk (col int) USING DELTA LOCATION 's3://trino-ci-test/databricks-compatibility-test-test_trino_preserves_versions_vtizu3eknquk'TBLPROPERTIES ('delta.minReaderVersion'='1', 'delta.minWriterVersion'='1', 'delta.checkpointInterval' = 1).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the explanation. Can it be captured as a code comment?

@ebyhr ebyhr mentioned this pull request Oct 19, 2022
@ebyhr ebyhr force-pushed the ebi/delta-fix-test branch from 3cb2431 to 9169110 Compare October 20, 2022 14:41
@ebyhr ebyhr added the no-release-notes This pull request does not require release notes entry label Oct 21, 2022
@ebyhr ebyhr merged commit d52cd58 into master Oct 21, 2022
@ebyhr ebyhr deleted the ebi/delta-fix-test branch October 21, 2022 07:53
@github-actions github-actions bot added this to the 401 milestone Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

2 participants