asynch icon indicating copy to clipboard operation
asynch copied to clipboard

Fix a bug in case the SimpleAggregateFunction type has a comma in its definition.

Open palkeo opened this issue 3 years ago • 2 comments

SimpleAggregateFunction(something, Tuple(String, String)) would trigger that bug as the inner variable would be Tuple(String.

This fixes it by splitting only on the first comma that it encounters. I assume there is no comma in the function name.

palkeo avatar Jun 30 '22 18:06 palkeo

Thanks! Could you add a unittest?

long2ice avatar Jul 01 '22 00:07 long2ice

Done.

palkeo avatar Jul 01 '22 15:07 palkeo