Skip to content

Added dfs.client.use.datanode.hostname config option#69

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
freshertm:add_datanode_hostname_option
Jan 9, 2025
Merged

Added dfs.client.use.datanode.hostname config option#69
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
freshertm:add_datanode_hostname_option

Conversation

@freshertm
Copy link
Copy Markdown
Contributor

Fixed #65

Added config option dfs.client.use.datanode.hostname support.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 10, 2024

CLA assistant check
All committers have signed the CLA.

@freshertm
Copy link
Copy Markdown
Contributor Author

@alexey-milovidov
I've added config option support.
Please review the PR. Thanks.

@alexey-milovidov alexey-milovidov self-assigned this Jan 9, 2025
@alexey-milovidov alexey-milovidov merged commit d0ae7d2 into ClickHouse:master Jan 9, 2025
@freshertm freshertm deleted the add_datanode_hostname_option branch January 14, 2025 11:21
@rschu1ze
Copy link
Copy Markdown
Member

rschu1ze commented Feb 2, 2025

It is neither clear what this PR nor what #65 is about but since HDFS is a community extension of ClickHouse and not supported in production envs (https://clickhouse.com/docs/en/engines/table-engines/integrations/hdfs), it's okay to merge.

@freshertm Consider contributing it back to upstream (though upstream looks completely dead).

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.

dfs.client.use.datanode.hostname have no effect

4 participants