liunaijie commented on code in PR #7928: URL: https://github.com/apache/seatunnel/pull/7928#discussion_r1821769526
########## docs/en/transform-v2/explode.md: ########## @@ -0,0 +1,67 @@ +# Explode + +> Explode transform plugin + +## Description + +Split a row of data into multiple data according to the specified delimiter. Review Comment: Hi, this code basic is looks good to me. But I have a thoughts about this explode feature, can we only support explode a list type column? the string split phase move to another phase, like add a sql function, `split(str_col, ';')` More over, we can add `explode` as a sql function, then the transform all use `Sql` transfrom. the sql is looks like `select explode(split(str_col,';')) from ` (Just discussion, WDYT @Hisoka-X ) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
