-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
[DOCS] ES|QL implicit casting #108618
[DOCS] ES|QL implicit casting #108618
Conversation
Documentation preview: |
Pinging @elastic/es-docs (Team:Docs) |
Pinging @elastic/es-analytical-engine (Team:Analytics) |
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 is looking good @fang-xing-esql, just a few suggestions for clarity and presentation so far :)
We would definitely want to link this page from the type conversion functions section, perhaps in a TIP
callout.
Thank you for reviewing Liam! The last commit is updated according to the comments, could you please take another look? |
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.
Couple minor things, plus let's add link in esql-language.asciidoc
, you already have the include
statement, but just need link text so it appears in the list on that page :)
Once addressed should be good to go, thank you!
docs/reference/esql/functions/type-conversion-functions.asciidoc
Outdated
Show resolved
Hide resolved
Thank you for double checking again @leemthompo ! All of the comments are addressed. |
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.
LGTM from a docs perspective. Thanks for iterating!
* implicit casting doc
💚 Backport successful
|
Add docs for implicit casting.