-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-6663] [SQL] use Literal.create instread of constructor #5320
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
Conversation
|
Test build #29577 has started for PR 5320 at commit |
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.
i think this line is now over 100 chars
|
LGTM as soon as you fix the style issues and jenkins pass. |
|
Test build #29577 has finished for PR 5320 at commit
|
|
Test FAILed. |
|
Test build #29578 has started for PR 5320 at commit |
|
Test FAILed. |
|
Test build #633 has started for PR 5320 at commit |
|
Test build #633 has finished for PR 5320 at commit
|
|
Thanks. Merging in master. |
In order to do inbound checking and type conversion, we should use Literal.create() instead of constructor.