Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add explicit location for the cloned table #21434

Merged

Conversation

findinpath
Copy link
Contributor

At the time of this writing, the metastore of the product test has the location for the default schema set to

hdfs://hadoop-master:9000/user/hive/warehouse

Use an explicit location for the cloned table in order to avoid writing the cloned table to hdfs.

Release notes

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

@cla-bot cla-bot bot added the cla-signed label Apr 5, 2024
@findinpath findinpath force-pushed the findinpath/delta-pt-location-fix branch from 0a0feb7 to f9bc5a7 Compare April 5, 2024 21:28
At the time of this writing, the metastore of the product test
has the location for the `default` schema set to

```
hdfs://hadoop-master:9000/user/hive/warehouse
```

Use an explicit location for the cloned table in order
to avoid writing the cloned table to hdfs.
@findinpath findinpath force-pushed the findinpath/delta-pt-location-fix branch from f9bc5a7 to e8b5a9c Compare April 5, 2024 21:35
@findinpath findinpath requested review from findepi and ebyhr April 5, 2024 21:36
@findepi
Copy link
Member

findepi commented Apr 5, 2024

/test-with-secrets sha=e8b5a9c019f6743823d2c6b50324bedfcab822e4

@ebyhr
Copy link
Member

ebyhr commented Apr 5, 2024

FYI, the affected test runs only on OSS Delta Lake.

Copy link

github-actions bot commented Apr 5, 2024

The CI workflow run with tests that require additional secrets has been started: https://github.com/trinodb/trino/actions/runs/8576260234

@findepi findepi merged commit 4bd0f8d into trinodb:master Apr 6, 2024
43 checks passed
@findepi findepi added test no-release-notes This pull request does not require release notes entry labels Apr 6, 2024
@github-actions github-actions bot added this to the 445 milestone Apr 6, 2024
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 test
Development

Successfully merging this pull request may close these issues.

3 participants