Skip to content

Union types documentation#110183

Merged
craigtaverner merged 8 commits intoelastic:mainfrom
craigtaverner:union_types_docs
Jul 16, 2024
Merged

Union types documentation#110183
craigtaverner merged 8 commits intoelastic:mainfrom
craigtaverner:union_types_docs

Conversation

@craigtaverner
Copy link
Copy Markdown
Contributor

Union types was added in #107545

This provides the associated documentation for this feature.

@craigtaverner craigtaverner added >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Jun 26, 2024
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added Team:Docs Meta label for docs team v8.15.0 labels Jun 26, 2024
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@craigtaverner craigtaverner requested a review from marciw June 26, 2024 10:33
----

....
Cannot use field [client_ip] due to ambiguities being mapped as [2] incompatible types:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to have a better styling for this error message. Right now this style does not really stand out sufficiently. Do we have a convention for displaying errors? A quick search did not find anything.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still looking into options and conventions, but for now I changed this to use [source,bash], which is kind of a default and looks decent (example). WDYT?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Certainly much better than my approach, although not yet quite perfect. Good enough for now.

Copy link
Copy Markdown
Contributor

@bpintea bpintea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@craigtaverner craigtaverner merged commit 1d6f1a0 into elastic:main Jul 16, 2024
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Jul 16, 2024
* Union types documentation

* Try remove asciidoc error

* Another attempt

* Using literal block

* Nicer formatting

* Remove partintro

* Small refinements

* Edits for clarity and style

---------

Co-authored-by: Marci W <333176+marciw@users.noreply.github.com>
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💚 Backport successful

Status Branch Result
8.15

elasticsearchmachine pushed a commit that referenced this pull request Jul 16, 2024
* Union types documentation

* Try remove asciidoc error

* Another attempt

* Using literal block

* Nicer formatting

* Remove partintro

* Small refinements

* Edits for clarity and style

---------

Co-authored-by: Marci W <333176+marciw@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Docs Meta label for docs team v8.15.0 v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants