-
Notifications
You must be signed in to change notification settings - Fork 908
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
Move Spark-indpendent Table debug to cudf Java #13783
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine. Just curious if we want to adjust/improve anything while we have the patient on the table?
Like passing in an output stream instead of always using stderr? Not that it is really needed.
I am open to these and further suggestions |
Signed-off-by: Gera Shegalov <gera@apache.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are many nit formatting issues. Please check carefully before merging.
I am going to hit reformat with |
Signed-off-by: Gera Shegalov <gera@apache.org>
/merge |
Description
spark-rapids has code for debugging JNI Tables/Columns that is useful for debugging during dev work in cudf
This PR proposes to start moving it to cudf/java. spark-rapids will be updated to call into the cudf in a follow-up PR.
Sample Usage with JShell
Checklist