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

YtOutputWriter should expose WriteTable#config #65

Conversation

faucct
Copy link
Collaborator

@faucct faucct commented Feb 13, 2025

Complementary to ytsaurus/ytsaurus#1019

@faucct faucct requested a review from alextokarew February 13, 2025 09:12
@@ -80,6 +80,8 @@ object YtTableSparkSettings {

case object SortColumns extends ConfigEntry[Seq[String]]("sort_columns", Some(Nil))

case object WriteTableConfig extends ConfigEntry[YTreeNode](s"write_table_config", Some(null))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Some(null) -> None

Copy link

robot-magpie bot commented Feb 13, 2025

@alextokarew has imported your pull request. If you are a Yandex employee, you can view this diff.

Copy link

robot-magpie bot commented Feb 14, 2025

✅ This pull request is being closed because it has been successfully merged into our internal monorepository.
Your changes will be pushed to this repository soon. Thank you for your contribution!

@robot-magpie robot-magpie bot closed this Feb 14, 2025
robot-piglet pushed a commit that referenced this pull request Feb 14, 2025
Complementary to ytsaurus/ytsaurus#1019

---

Pull Request resolved: #65
commit_hash:94646d99ec4d87375d18f7dc849826cd2f092e92
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