Skip to content

Conversation

DavidKatz-il
Copy link
Contributor

file_type now supports the following types ['csv', 'jdbc', 'json', 'orc', 'parquet', 'saveAsTable', 'text'].
file_options now supports all existing params.

Copy link
Member

@schrockn schrockn left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks a bunch for this

@schrockn
Copy link
Member

Some tests would be good too...

@DavidKatz-il DavidKatz-il requested a review from schrockn June 18, 2020 22:43
Copy link
Member

@schrockn schrockn left a comment

Choose a reason for hiding this comment

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

looks good. would be awesome to have final, minor comments addressed. thanks so much!

@schrockn schrockn merged commit 3f55ed4 into dagster-io:master Jun 23, 2020
@DavidKatz-il DavidKatz-il deleted the add-options-spark-df branch June 25, 2020 10:02
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