Skip to content

Moves ES|QL out of Technical preview (GA) - #178860

Merged
stratoula merged 13 commits into
elastic:mainfrom
stratoula:esql-out-of-ga
Apr 9, 2024
Merged

stratoula merged 13 commits into
elastic:mainfrom
stratoula:esql-out-of-ga

Conversation

@stratoula

@stratoula stratoula commented Mar 18, 2024 •

Copy link
Copy Markdown
Contributor

Summary

Moves ES|QL out of Technical preview (GA) which means that:

  • Removes tech preview badge from unified search
  • Removes tech preview callout from Discover
  • Removes tech preview badge from inline editing flyout (I am keeping it to indicate the inline editing is still on tech preview but I am removing the specific ES|QL message
  • Removes tech preview text from advanced setting

@stratoula stratoula added Feature:ES|QL ES|QL related features in Kibana release_note:breaking backport:skip This PR does not require backporting v8.14.0 labels Mar 18, 2024
@nik9000

nik9000 commented Mar 18, 2024

Copy link
Copy Markdown
Contributor

out of GA

out of tech preview?

@stratoula stratoula changed the title Moves ES|QL out of GA Moves ES|QL out of Technical preview Mar 18, 2024
@stratoula stratoula changed the title Moves ES|QL out of Technical preview Moves ES|QL out of Technical preview (GA) Mar 18, 2024
@stratoula

Copy link
Copy Markdown
Contributor Author

/ci

@stratoula stratoula added the Team:ESQL ES|QL related features in Kibana t// label Apr 2, 2024
@stratoula
stratoula marked this pull request as ready for review April 2, 2024 07:20
@stratoula
stratoula requested review from a team as code owners April 2, 2024 07:20
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@dej611 dej611 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Visualization changes 👍

@jughosta jughosta left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Discover changes LGTM 👍

Can we also rephrase "Try ES|QL" to something else?

Screenshot 2024-04-02 at 10 14 21

@stratoula

Copy link
Copy Markdown
Contributor Author

@jughosta let me ask the group.

I am going to merge this PR closer to the FF, thanx all for your reviews

@jughosta

jughosta commented Apr 2, 2024

Copy link
Copy Markdown
Contributor

@stratoula Thanks! Linking to #176291 as it would need to be updated there too.

@stratoula

Copy link
Copy Markdown
Contributor Author

@jughosta I think we are ok with the text for now, but thanx for bringing this to our attention.

@stratoula

Copy link
Copy Markdown
Contributor Author

/ci

@kibana-ci

Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
discover 778 777 -1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
discover 591.0KB 590.4KB -679.0B
lens 1.4MB 1.4MB -175.0B
unifiedSearch 225.5KB 225.4KB -164.0B
total -1018.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@stratoula
stratoula merged commit 49777dc into elastic:main Apr 9, 2024
@drewdaemon

Copy link
Copy Markdown
Contributor

ES|QL is free!!!! 🦋🦋🦋🦋

@ninoslavmiskovic

Copy link
Copy Markdown
Contributor

Huge accomplishment @stratoula 👏👏👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:ES|QL ES|QL related features in Kibana release_note:breaking Team:ESQL ES|QL related features in Kibana t// v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants