Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'group()' built-in function to DaphneDSL. #921

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Nov 20, 2024

  1. - Add 'group()' built-in function to DaphneDSL.

     - This built-in function creates a GroupOp in DaphneIR.
     - Only support 'SUM' as an aggregation function.
     - Get only one aggregation column.
     - Get an arbitrary number of columns to group on.
    - Add support for string values in the 'group' kernel function.
      - SUM, MIN, and MAX are the only aggregation functions applied to string columns.
      - Other aggregation functions throw an exception if they receive strings as arguments or results.
      - Additionally, 'DeduceValueTypeAndExecute' cannot handle string values due to unsupported operations on strings.
      - Therefore, 'ColumnGroupAggStringVTArg' that is specialized for strings is used.
      - Or 'ColumnGroupAgg' is called exclusively with string values.
    - The 'group' function internally calls the 'order' and 'extractCol' kernel functions.
      - These two functions are updated to handle string values correctly.
    - Added script-level test cases to validate the new functionality.
    - Close issue daphne-eu#903
    saminbassiri committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    02dd2cf View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    d223d16 View commit details
    Browse the repository at this point in the history