-
Notifications
You must be signed in to change notification settings - Fork 41
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
Write directly to HDFS #24
Comments
To clarify, is your patch to expand the generator to be able to write to an hdfs file instead of a regular file? That seems reasonable. We'd be happy to take a PR. |
Yes, that's correct -- it's pretty simple frankly.
I just need to do some cleanup in your pom as some of stuff was black
listed IIRC.
On Tue, Aug 29, 2017 at 9:43 AM Rebecca Schlussel ***@***.***> wrote:
To clarify, is your patch to expand the generator to be able to write to
an hdfs file instead of a regular file? That seems reasonable. We'd be
happy to take a PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#24 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AACHV2AyvfvFW50bGlrmiAt3iYH5aIbmks5sdD-ygaJpZM4PGDNi>
.
--
snet from a itny keybaord
|
mcdan
added a commit
to mcdan/tpcds
that referenced
this issue
Aug 30, 2017
* Make session & scaling serializable so it can be passed around * Teach session factory pattern to take in a hadoop config object * Make a serializable hadoop config object to ensure Session is serializable
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there any interest in teaching this library to also write directly to HDFS connected storage? I have a patch that handles it already but before I go to clean up the code and make it presentable I wanted to see if you would be at all interested in a feature like that?
The text was updated successfully, but these errors were encountered: