This is an automated email from the ASF dual-hosted git repository.
shenyi pushed a commit to branch aggregate-transform
in repository https://gitbox.apache.org/repos/asf/echarts.git
The following commit(s) were added to refs/heads/aggregate-transform by this
push:
new aa700ef feat(transform): use val directly in groupBy dimension result
aa700ef is described below
commit aa700eff62bf288b4b8b51fcfa7e6fba600485f2
Author: pissang <[email protected]>
AuthorDate: Wed Mar 23 22:26:53 2022 +0800
feat(transform): use val directly in groupBy dimension result
---
src/component/transform/aggregateTransform.ts | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/src/component/transform/aggregateTransform.ts
b/src/component/transform/aggregateTransform.ts
index 93e8d9f..cd32c77 100644
--- a/src/component/transform/aggregateTransform.ts
+++ b/src/component/transform/aggregateTransform.ts
@@ -252,18 +252,12 @@ function doAggregate(
) {
function doCreate(isGroupByDim: boolean, aggResult: AggregateResult, val:
OptionDataValue, groupByVal?: string) {
- aggResult.set(groupByVal, isGroupByDim
- ? groupByVal
- : methods[aggResult.method].init(val, aggResult, groupByVal)
- );
+ aggResult.set(groupByVal, isGroupByDim ? val :
methods[aggResult.method].init(val, aggResult, groupByVal));
};
function doUpdate(aggResult: AggregateResult, val: OptionDataValue,
groupByVal?: string) {
const method = methods[aggResult.method];
if (method.add) {
- aggResult.set(
- groupByVal,
- method.add(aggResult.get(groupByVal), val, aggResult,
groupByVal)
- );
+ aggResult.set(groupByVal, method.add(aggResult.get(groupByVal),
val, aggResult, groupByVal));
}
};
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
