-
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
[SQL][Docs] updated the documentation for explode #6943
Conversation
the syntax was incorrect in the example in explode
ok to test. |
LGTM. Pending Jenkins. |
Jenkins, ok to test. |
Test build #35519 has finished for PR 6943 at commit
|
Does |
@srowen
I didn't think that I needed a JIRA ticket. The JIRA the documentation stated "However, trivial changes, where the what should change is virtually the same as the how it should change do not require a JIRA. Example: "Fix typos in Foo scaladoc"" So in the future I should put the [Docs] in the pull request? Is that what you meant? |
Yes, seems OK to me. Just put |
the syntax was incorrect in the example in explode Author: lockwobr <[email protected]> Closes #6943 from lockwobr/master and squashes the following commits: 3d864d1 [lockwobr] updated the documentation for explode (cherry picked from commit 4f7fbef) Signed-off-by: Kousuke Saruta <[email protected]>
the syntax was incorrect in the example in explode Author: lockwobr <[email protected]> Closes apache#6943 from lockwobr/master and squashes the following commits: 3d864d1 [lockwobr] updated the documentation for explode (cherry picked from commit 4f7fbef) Signed-off-by: Kousuke Saruta <[email protected]>
the syntax was incorrect in the example in explode