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

Update configuration file #372

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Update configuration file #372

merged 1 commit into from
Dec 9, 2024

Conversation

mzitnik
Copy link
Collaborator

@mzitnik mzitnik commented Dec 9, 2024

Summary

Update configuration file

Checklist

Delete items not relevant to your PR:

@mzitnik mzitnik merged commit 2f86084 into main Dec 9, 2024
30 of 36 checks passed
mahdimalverdi added a commit to mahdimalverdi/spark-clickhouse-connector that referenced this pull request Dec 21, 2024
* Update configurtion file (ClickHouse#372)

* Support ClickHouse insert settings for table writes (ClickHouse#369)

- Enable custom ClickHouse insert settings when writing to tables.
- Add support for `spark.clickhouse.write.settings.*` configuration.
- Ensure settings are applied to the `INSERT INTO ... SETTINGS ...` SQL command.
- Update documentation to describe usage of write settings.
- Add unit tests to validate behavior.

Closes ClickHouse#369

* Support ClickHouse insert settings for table writes (ClickHouse#369)

- Enable custom ClickHouse insert settings when writing to tables.
- Add support for `spark.clickhouse.write.settings` configuration.
- Update documentation to describe usage of write settings.

Closes ClickHouse#369

* reformat the codes

Closes ClickHouse#369

---------

Co-authored-by: Mark Zitnik <[email protected]>
Co-authored-by: Mahdi Malverdi <[email protected]>
mahdimalverdi pushed a commit to mahdimalverdi/spark-clickhouse-connector that referenced this pull request Dec 21, 2024
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.

2 participants