Native support for STRING_AGG

[ryanmle2001]

Related to issue #1331

Process:

• Added STRING_AGG to evadb.lark using LARK grammar per EvaDB new command documentation
• Define new ExpressionType. AGGREGATION_STRING_AGG enum
• Add AGGREGATION_STRING_AGG to aggregation_expression evaluate()
• Define STRING_AGG aggregation in Batch class with error handlers for 1) invalid column input and 2) invalid delimiter
• Unit tests

[xzdandy]

Could you revert the test.sh change?

[xzdandy]

I think the access the permission is still changed. 100644 → 100755

[xzdandy]

Thanks for adding the unit tests they look good. I think we also need an integration test where we have the STRING_AGG in a SQL query to verify that it works from end to end.

[ryanmle2001]

Thanks for adding the unit tests they look good. I think we also need an integration test where we have the STRING_AGG in a SQL query to verify that it works from end to end.

Can you send some code pointers on writing an integration test here?

[xzdandy]

Thanks for adding the unit tests they look good. I think we also need an integration test where we have the STRING_AGG in a SQL query to verify that it works from end to end.

Can you send some code pointers on writing an integration test here?

@ryanmle2001 Here

evadb/test/integration_tests/long/test_huggingface_functions.py

Line 412 in bbfa483

def test_select_and_groupby_with_paragraphs(self):

is an example, replace the SEGMENT with the STRING_AGG