sitespeedio / sitespeedio/sitespeed.io

InfluxDB - groupSeparator is only applied to one summaryType

Open
#2,358 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
JavaScript
Stars
5k
Forks
624
Avg merge
4h 8m
Merged PRs (30d)
2

Description

Sitespeed.io 8.6.5

"influxdb": { ... "groupSeparator": ".",

When this config option is set, only the Summary type is formatted correctly, as I query the data in InfluxDB.

time                browser category connectivity group                       max  mdev mean median min  origin      p10  p90  p99  page            pageTimings  statistics summaryType timings
----                ------- -------- ------------ -----                       ---  ---- ---- ------ ---  ------      ---  ---  ---  ----            -----------  ---------- ----------- -------
1551820685272654000 chrome  default  native       performance2_rc_nextiva_xyz 1733 0    1733 1733   1733 browsertime 1733 1733 1733 Launch_Platform pageLoadTime timings    pageSummary 
1551820685348698000 chrome  default  native       performance2_rc_nextiva_xyz 6193 0    6193 6193   6193 browsertime 6193 6193 6193 Login           pageLoadTime timings    pageSummary 
1551820685399924000 chrome  default  native       performance2_rc_nextiva_xyz 2918 0    2918 2918   2918 browsertime 2918 2918 2918 Open_CRM        pageLoadTime timings    pageSummary 
1551820685449569000 chrome  default  native       performance2.rc.nextiva.xyz 6193      3615 2918   1733 browsertime      6193                                              summary     pageTimings

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the InfluxDB output using the shown groupSeparator configuration and compare the formatting for pageSummary and summary records. Trace the InfluxDB integration entry point responsible for writing these summary types; done means the separator is applied consistently across both summary types.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
databases
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.