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

CSV report gets printed with duplicate ids #140446

Closed
bhavyarm opened this issue Sep 9, 2022 · 11 comments · Fixed by #143807
Closed

CSV report gets printed with duplicate ids #140446

bhavyarm opened this issue Sep 9, 2022 · 11 comments · Fixed by #143807
Labels
bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:medium Medium Level of Effort

Comments

@bhavyarm
Copy link
Contributor

bhavyarm commented Sep 9, 2022

Kibana version: 8.4.0

Elasticsearch version: 8.4.0

Server OS version: darwin_x86_64

Browser version: chrome latest

Browser OS version: OS X

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

Describe the bug: Kibana's discover is printing the same ids in CSV report.

Ingest:

PUT _template/template_test_elastic
{
  "index_patterns": [
    "prueba_elastic*"
  ],
  "settings": {
    "number_of_shards": 1
  },
  "mappings": {
    "properties": {
      "TIPO_MENSAJE": {
        "type": "keyword"
      },
      "DATE_TIME": {
        "type": "date",
        "format": "yyyyMMddHHmmss||yyyyMMddHHmmssZ||strict_date_optional_time||epoch_millis"
      }
    }
  }
}
 
POST prueba_elastic/_doc/202209071000000604
{
  "DATE_TIME": "20200101130000+0200",
  "TIPO_MENSAJE": "OP"
}

POST prueba_elastic/_doc/202209071000000605
{
  "DATE_TIME": "20200101130000+0200",
  "TIPO_MENSAJE": "OP"
}

POST prueba_elastic/_doc/202209071000000606
{
  "DATE_TIME": "20200101130000+0200",
  "TIPO_MENSAJE": "OP"
}

POST prueba_elastic/_doc/202209071000000607
{
  "DATE_TIME": "20200101130000+0200",
  "TIPO_MENSAJE": "OP"
}
  1. Create a data view with test_elastic index pattern
  2. Go to discover
  3. print csv report

Kibana prints duplicate ids in the csv report

Screen Shot 2022-09-09 at 3 33 27 PM

@bhavyarm bhavyarm added bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:Reporting Services Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph) labels Sep 9, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@elasticmachine
Copy link
Contributor

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

@kertal
Copy link
Member

kertal commented Sep 12, 2022

our CSV generation is assuming in the given case that the id is a number , and therefore it's converted to a number. However the number is too big and this causes rounding errors (#40183), which lead to duplicate ids. So a possible solution could be for CSV generation to consider the actual mapping of the field to determine if it's a number, or not. A quicker but, intermediate solution could be to always consider _id fields as string.

@kertal kertal removed the Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph) label Sep 12, 2022
@exalate-issue-sync exalate-issue-sync bot added the impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. label Sep 12, 2022
@exalate-issue-sync exalate-issue-sync bot added the loe:small Small Level of Effort label Sep 23, 2022
@Ning2022
Copy link

@LeeDr, Hi Lee, this is Ning, APAC support manager. I noticed you are Kibana SDH escalation contact point this week based on https://elastic.pagerduty.com/schedules#PNAOA6T, so I am contacting you here to get your insights. The customer is chasing the ETA for this issue through AE and CSM. Could you help review and share our position and plan on this issue for reducing customer's anxiety? Thanks a lot!

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServicesSv)

@LeeDr
Copy link
Contributor

LeeDr commented Sep 27, 2022

Hi @Ning2022. We were under the impression that a work-around was provided to the customer and so this wasn't a priority. Can you please update us on the success of using the work-around?

@LeeDr
Copy link
Contributor

LeeDr commented Sep 27, 2022

/cc @italo-cocio Any news on the customer using the work-around for this issue?

@italo-cocio
Copy link

Hello @LeeDr

My apologies for this, I thought somehow this piece of info was already in the know of our Dev team.
Customer mentions that the workaround involves a lot of changes in their application and it may cause performance issues, so they are not implementing it.

Thanks

@tsullivan
Copy link
Member

Update for the watchers on this issue: it's a priority for the team and we plan to release a fix in the next minor release.

@tsullivan
Copy link
Member

accidentally closed when I meant just to comment

@tsullivan tsullivan reopened this Sep 27, 2022
@exalate-issue-sync exalate-issue-sync bot added impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. and removed impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. labels Sep 27, 2022
@Ning2022
Copy link

@tsullivan , Hi Tim, thanks a lot for your update! Great to know we plan to fix it in the next minor release.

@exalate-issue-sync exalate-issue-sync bot added loe:medium Medium Level of Effort and removed loe:small Small Level of Effort labels Oct 24, 2022
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 (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:medium Medium Level of Effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants