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

Qubole Hook Does Not Support 'include_headers' #15598

Closed
levyitay opened this issue Apr 29, 2021 · 3 comments · Fixed by #15615
Closed

Qubole Hook Does Not Support 'include_headers' #15598

levyitay opened this issue Apr 29, 2021 · 3 comments · Fixed by #15615

Comments

@levyitay
Copy link
Contributor

Description

Qubole Hook and Operator do not support include_header param for getting results with headers
Add Support for include_header get_results(... arguments=[True])
Use case / motivation

It's very hard to work with CSV results from db without headers.
This is super important when using Qubole's databases.

Are you willing to submit a PR?

Not sure yet, I can give it a try

Related Issues

@levyitay levyitay added the kind:feature Feature Requests label Apr 29, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented Apr 29, 2021

Thanks for opening your first issue here! Be sure to follow the issue template!

@eladkal
Copy link
Contributor

eladkal commented Apr 30, 2021

@levyitay assigned to you

@levyitay
Copy link
Contributor Author

@eladkal Can you take a look and push it if possible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants