You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Feedback on top_metrics UI in transforms (added in 7.14).
select a better parent name
From the UI, we try to use a dest field naming convention that groups fields together. E.g. CPU.min and CPU.max are both children of CPU . The UI workflow takes the initial selected field to use as the parent name.
Now top_metrics is available, this creates weird filenames as the initial field selected might not logically be a sensible parent name.
Current implementation will causes top metrics fields, depending on whatever was selected first, such as phone_no.top_metrics.last_name and DiskReadBytes.top_metrics.CPUUtilization .
To workaround, it can be renamed, but this is not ideal because it seems like it will always need to be renamed.
Suggest (short term):
top_metrics becomes the parent. e.g. top_metrics.last_name and top_metrics.phone_no
If multiple separate top_metrics have been selected, then increment the additional ones (top_metrics1 and top_metrics2)
removing advanced settings
Advanced settings exist for configuring sorting of array fields. This only become visible after selecting the sort field and sort order. Both of these are required, therefore the adv settings are always displayed at the end.
It is unlikely that sorting on arrays is required and therefore it should be removed from the UI in order to simplify. As with other niche settings, it should still be possible to configure using the JSON editor.
select default soft fields
The user should be able to configure a preferred sort field and direction. However in many cases, there will only be 1 element and sorting will not make a difference.
Suggest keeping the ability for user to pick their own sort fields, but picking a sensible default so they do not always have to click and select. If a time field has been included in the group by or aggregation, pick this and sort by most recent. If there are multiple time fields, then choose the first one included in the group by. Otherwise leave it for the user to select their own sort field, with a default of desc.
for info: configuring top_metrics.size
Transforms will only return the first element even if a user configures size to be a value greater than 1 using the JSON editor. There is no protection code that stops the user from setting it to another value. More discussion here elastic/elasticsearch#74420. Including for reference and for visibility. However no actions required.
Feedback on
top_metricsUI in transforms (added in 7.14).From the UI, we try to use a dest field naming convention that groups fields together. E.g.
CPU.minandCPU.maxare both children ofCPU. The UI workflow takes the initial selected field to use as the parent name.Now
top_metricsis available, this creates weird filenames as the initial field selected might not logically be a sensible parent name.Current implementation will causes top metrics fields, depending on whatever was selected first, such as
phone_no.top_metrics.last_nameandDiskReadBytes.top_metrics.CPUUtilization.To workaround, it can be renamed, but this is not ideal because it seems like it will always need to be renamed.
Suggest (short term):
top_metricsbecomes the parent. e.g.top_metrics.last_nameandtop_metrics.phone_noIf multiple separate
top_metricshave been selected, then increment the additional ones (top_metrics1andtop_metrics2)removing advanced settings
Advanced settings exist for configuring sorting of array fields. This only become visible after selecting the sort field and sort order. Both of these are required, therefore the adv settings are always displayed at the end.
It is unlikely that sorting on arrays is required and therefore it should be removed from the UI in order to simplify. As with other niche settings, it should still be possible to configure using the JSON editor.
The user should be able to configure a preferred sort field and direction. However in many cases, there will only be 1 element and sorting will not make a difference.
Suggest keeping the ability for user to pick their own sort fields, but picking a sensible default so they do not always have to click and select. If a time field has been included in the group by or aggregation, pick this and sort by most recent. If there are multiple time fields, then choose the first one included in the group by. Otherwise leave it for the user to select their own sort field, with a default of
desc.top_metrics.sizeTransforms will only return the first element even if a user configures
sizeto be a value greater than 1 using the JSON editor. There is no protection code that stops the user from setting it to another value. More discussion here elastic/elasticsearch#74420. Including for reference and for visibility. However no actions required.