Skip to content
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

[Discover][Filter by field type] Toggle to hide empty fields has disappeared #162239

Closed
oshmyrko opened this issue Jul 19, 2023 · 3 comments
Closed
Labels
bug Fixes for quality problems that affect the customer experience Feature:Discover Discover Application Feature:UnifiedFieldList The unified field list component used by Lens & Discover Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. v8.8.2

Comments

@oshmyrko
Copy link

Kibana version: 8.8.2

Elasticsearch version: 8.8.2

Server OS version: Ubuntu 20.04.6 LTS

Browser version: Firefox 114.0.2, Chrome 114.0.5735.198

Browser OS version: MacOS 12.5.1 (Monterey)

Original install method (e.g. download page, yum, from source, etc.): ECK operator

Describe the bug:

The toggle to hide empty fields on the left panel of the Discover page has disappeared from Filter by field type after version 8.6.1.

Steps to reproduce:

  1. Open the Discover page
  2. Try to hide fields from the left panel that have no values (value not set)

Expected behavior:

Filter by field type has the toggle to hide empty fields, so that fields that do not have values (value not set) can be hidden.

Screenshots (if relevant):

  • Kibana 8.6.1
    image

  • Kibana 8.8.2
    image

Any additional context:
We are using one general mapping in our index templates to index incoming logs, but since some indices do not contain values for some fields, we would like to hide such fields from the left panel.
Here is an example with IP address field which I would like to hide as well:
image

@oshmyrko oshmyrko added the bug Fixes for quality problems that affect the customer experience label Jul 19, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label Jul 19, 2023
@oshmyrko oshmyrko changed the title [Discover] Toggle to hide empty fields has disappeared from filter by field type [Discover][Filter by field type] Toggle to hide empty fields has disappeared Jul 19, 2023
@oshmyrko
Copy link
Author

Seems related to #147124.

@davismcphee davismcphee added Feature:Discover Discover Application Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. Feature:UnifiedFieldList The unified field list component used by Lens & Discover v8.8.2 and removed needs-team Issues missing a team label labels Jul 19, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@davismcphee
Copy link
Contributor

davismcphee commented Jul 19, 2023

@oshmyrko This is indeed related to #147124, and a little background on this issue:

The change to remove the "Hide empty fields" toggle was intentional as part of an effort to unify the field lists between Discover and Lens as detailed in #135678 (with the actual work completed in #144412).

This was done not only to improve consistency between the apps, but also because the "Hide empty fields" toggle never really functioned as the label would imply. Rather than hiding fields that would be empty based on the query and time range, the "Hide empty fields" feature would simply loop through the 500 document sample returned to Discover and hide any fields for which it couldn't find values within the sample documents. This misled users into believing these fields were empty for all documents returned by their query, when in reality the 501st document could contain values for all "empty" fields, but the fields would still be hidden in the field list.

To account for this we adopted the pattern Lens uses of including an "Empty fields" section into our sidebar which should incude all fields that don't contain values based on the current query and time range. Unfortunately this feature too does not work as intended due to limitations in the upstream Elasticsearch API we rely on -- that's where #147124 comes into play.

I'm going to close this as a duplicate of #147124, but we understand how frustrating not having this functionality can be for users, and we're currently working with the Elasticsearch team directly to figure out how to provide proper support for this feature. Many apologies for the inconvenience here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Discover Discover Application Feature:UnifiedFieldList The unified field list component used by Lens & Discover Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. v8.8.2
Projects
None yet
Development

No branches or pull requests

3 participants