Skip to content

Improve docs for support hadoop compatible file system when use HDFS …#25452

Closed
hqbhoho wants to merge 1 commit into
trinodb:masterfrom
hqbhoho:feature/improve_docs_for_exchange_hdfs_v2
Closed

Improve docs for support hadoop compatible file system when use HDFS …#25452
hqbhoho wants to merge 1 commit into
trinodb:masterfrom
hqbhoho:feature/improve_docs_for_exchange_hdfs_v2

Conversation

@hqbhoho
Copy link
Copy Markdown
Contributor

@hqbhoho hqbhoho commented Mar 29, 2025

Description

Follow up to #25448
While exchange FileSystem is a singleton instance which use fs.defaultFS in core-site.xml as default uri, so when use other hadoop-compatible file system,we should config fs.defaultFS to match hadoop-compatible file system. eg: we wan't use oss, the fs.defaultFS would be configured as oss://xxxx in core-site.xml. Therefore, I think we should add some hints in the documentation.

Additional context and related issues

Release notes

( ) 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 Mar 29, 2025
@github-actions github-actions Bot added the docs label Mar 29, 2025
@hqbhoho hqbhoho requested a review from mosabua March 29, 2025 06:28

1. Configure the `AbstractFileSystem` implementation in `core-site.xml`.
2. Add the relevant client JAR files into the directory `${Trino_HOME}/plugin/exchange-hdfs`
1. Configure the `fs.defaultFS` with specific scheme and
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.

What do you you mean by that .. maybe add an example ? Also fyi @losipiuk ?

@hqbhoho hqbhoho requested a review from losipiuk March 31, 2025 00:59
@github-actions
Copy link
Copy Markdown

This pull request has gone a while without any activity. Ask for help on #core-dev on Trino slack.

@github-actions github-actions Bot added the stale label Apr 21, 2025
@hqbhoho hqbhoho closed this Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants