fix sum avg alignment (#172)

liujiayi771 · web-flow · commit 4af2ca61d2a6 · 2023-03-31T15:37:44.000+08:00
diff --git a/velox/functions/sparksql/aggregates/DecimalAvgAggregate.h b/velox/functions/sparksql/aggregates/DecimalAvgAggregate.h
@@ -36,7 +36,7 @@ class DecimalAverageAggregate : public exec::Aggregate {
   }
 
   int32_t accumulatorAlignmentSize() const override {
-    return sizeof(LongDecimalWithOverflowState);
+    return static_cast<int32_t>(sizeof(int128_t));
   }
 
   void initializeNewGroups(
diff --git a/velox/functions/sparksql/aggregates/DecimalSumAggregate.h b/velox/functions/sparksql/aggregates/DecimalSumAggregate.h
@@ -42,7 +42,7 @@ class DecimalSumAggregate : public exec::Aggregate {
   }
 
   int32_t accumulatorAlignmentSize() const override {
-    return sizeof(DecimalSum<TAccumulator>);
+    return static_cast<int32_t>(sizeof(TAccumulator));
   }
 
   void initializeNewGroups(

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class DecimalAverageAggregate : public exec::Aggregate {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`int32_t accumulatorAlignmentSize() const override {`
`39`		`- return sizeof(LongDecimalWithOverflowState);`
	`39`	`+ return static_cast<int32_t>(sizeof(int128_t));`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`void initializeNewGroups(`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ class DecimalSumAggregate : public exec::Aggregate {`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`int32_t accumulatorAlignmentSize() const override {`
`45`		`- return sizeof(DecimalSum<TAccumulator>);`
	`45`	`+ return static_cast<int32_t>(sizeof(TAccumulator));`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`void initializeNewGroups(`