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

[Feat] Use metric column as prefix when returning multiple value columns #12727

Closed
marefr opened this issue Jul 26, 2018 · 3 comments
Closed

Comments

@marefr
Copy link
Member

marefr commented Jul 26, 2018

Currently if you return a metric column and multiple value columns all value columns will be munged into the same series. Would be nice if the metric column could be used as prefix for the value columns.

@marefr
Copy link
Member Author

marefr commented Jul 30, 2018

Fixed by #12679

@marefr marefr closed this as completed Jul 30, 2018
marefr added a commit that referenced this issue Jul 30, 2018
ryantxu added a commit to NatelEnergy/grafana that referenced this issue Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 2, 2018
* grafana/master: (185 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 2, 2018
* grafana/master: (263 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
@marefr marefr modified the milestones: 5.3, 5.3.0-beta1 Sep 6, 2018
@beingyash
Copy link

@marefr This also needs to be available for elasticsearch as when multiple queries return the same series names, the transformation operations cannot distinguish the series returned by both queries individually and hence show only the series names once.

@marefr
Copy link
Member Author

marefr commented Jan 15, 2021

@beingyash please open a separate issue to track if there isn't one already open.

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

No branches or pull requests

2 participants