-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
[SPARK-3705][SQL]add case for VoidObjectInspector to cover NullType #2552
Conversation
Can one of the admins verify this patch? |
ok to test |
QA tests have started for PR 2552 at commit
|
Tests timed out after a configured wait of |
Test FAILed. |
@marmbrus , it seems all PR of SQL tests timed out |
I tested for the time out issue, #1689 lead to this issue, but have not found the root cause |
Sorry, I've been really busy and haven't had time to review PRs. Starting tests now. Pending those this LGTM. |
QA tests have started for PR 2552 at commit
|
QA tests have finished for PR 2552 at commit
|
Thanks! Merged to master. |
add case for VoidObjectInspector in
inspectorToDataType