From 79b0e6e0cf448b188cc507e191d213375a17b7ef Mon Sep 17 00:00:00 2001
From: Yilin Chen <sticnarf@gmail.com>
Date: Wed, 29 May 2019 16:09:11 +0800
Subject: [PATCH] Update with new parse interface

Signed-off-by: Yilin Chen <sticnarf@gmail.com>
---
 src/coprocessor/dag/aggr_fn/impl_first.rs | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/coprocessor/dag/aggr_fn/impl_first.rs b/src/coprocessor/dag/aggr_fn/impl_first.rs
index bc197855deaf..0471f00eaad8 100644
--- a/src/coprocessor/dag/aggr_fn/impl_first.rs
+++ b/src/coprocessor/dag/aggr_fn/impl_first.rs
@@ -18,24 +18,14 @@ pub struct AggrFnDefinitionParserFirst;
 impl super::AggrDefinitionParser for AggrFnDefinitionParserFirst {
     fn check_supported(&self, aggr_def: &Expr) -> Result<()> {
         assert_eq!(aggr_def.get_tp(), ExprType::First);
-        if aggr_def.get_children().len() != 1 {
-            return Err(box_err!(
-                "Expect 1 parameter, but got {}",
-                aggr_def.get_children().len()
-            ));
-        }
-
-        // Check whether parameter expression is supported.
-        RpnExpressionBuilder::check_expr_tree_supported(&aggr_def.get_children()[0])?;
-
-        Ok(())
+        super::util::check_aggr_exp_supported_one_child(aggr_def)
     }
 
     fn parse(
         &self,
         mut aggr_def: Expr,
         time_zone: &Tz,
-        max_columns: usize,
+        src_schema: &[FieldType],
         out_schema: &mut Vec<FieldType>,
         out_exp: &mut Vec<RpnExpression>,
     ) -> Result<Box<dyn super::AggrFunction>> {