Node map agg #1160

andyfengHKU · 2023-01-08T13:23:33Z

This PR allows returning node/rel as all of their properties aggregated in json format in a single column. To do so, this PR

Add BoundStatementResult which defines the schema of how to aggregate multiple columns into a single column in json format
Move ownership of iterate tuple from iterator to query result. So iterator only writes to a vector of flat values.

Move isReadOnly, isDDL, isCopyCSV check from planning stage to binding stage
Alter preparedStatement to take a vector of logical plans and thus remove APIs exposed only for testing framework

test/test_files/tinysnb/projection/single_label.test

src/include/binder/binder.h

src/include/binder/bound_statement_result.h

src/include/main/prepared_statement.h

src/include/main/query_result.h

src/include/processor/mapper/plan_mapper.h

src/include/processor/physical_plan.h

src/include/processor/result/factorized_table.h

src/include/processor/result/value.h

andyfengHKU requested a review from ray6080 January 8, 2023 13:23

andyfengHKU marked this pull request as ready for review January 8, 2023 13:23

ray6080 approved these changes Jan 9, 2023

View reviewed changes

return node,rel as json format

83cd251

andyfengHKU force-pushed the node-map-agg branch from 25daa9b to 83cd251 Compare January 9, 2023 06:53

andyfengHKU merged commit 70fc9e1 into master Jan 9, 2023

andyfengHKU deleted the node-map-agg branch January 9, 2023 07:09

Provide feedback