Carson Ip

Results 98 comments of Carson Ip

> CI because there's a shortcut for docs to bypass the lint check actually, the same bypass exists for 8.7 branch, but because there are `.yml` files deleted in the...

This needs to be tested thoroughly, as my experiments show that the memory profiles (v2 vs v4) look quite different: v2: https://github.com/carsonip/tbs-badger-playground/tree/main/prefetch v4: https://github.com/carsonip/tbs-badger-playground/tree/badgerv4/prefetch

## Existing implementation Transaction metrics contains the dimensions of ServiceAggregationKey + ServiceInstanceAggregationKey + TransactionAggregationKey. ```go type ServiceAggregationKey struct { Timestamp time.Time ServiceName string ServiceEnvironment string ServiceLanguageName string AgentName string }...

## Revised proposal (updated after [Andrew's comment](https://github.com/elastic/apm-server/issues/11266#issuecomment-1652790217)) Existing transaction metrics will be updated to have no instance-specific dimensions and instance-specific dimensions will go into a new metrics, which is called...

I would like to note that the changes to aggregation will be made in apm-aggregation, not existing aggregation code in apm-server. apm-server will use apm-aggregation after #11117 which is also...

> What's the rationale behind not having the cloud metadata in transaction metrics? Are we expecting the combination of transaction name and regions/availability zones to create too high cardinality? I...

Thanks for confirming. I'll push forward with the revised proposal [in my previous comment](https://github.com/elastic/apm-server/issues/11266#issuecomment-1651432073).

Would like a confirmation about an implementation detail about service instance transaction metrics overflow bucket. Are we also using `transaction.type: _other` as overflow key for service instance transaction metrics overflow?...

> I am not sure if i understand the question correctly. We were using `transaction.name: _other` and `service.name: _other` for transaction and service metrics. @carsonip Are you proposing to use...

@gbamparop mentioned in https://github.com/elastic/kibana/issues/162392#issuecomment-1662364252 that the APM UI counterpart may miss 8.10.