Delete node/rel properties without transactions #1169

acquamarin · 2023-01-11T05:43:37Z

This PR adds support for deleting node/rel propties. (Without supporting transaction).
Refactors the copy_csv/drop_table operators.
Breaks the assumption that the propertyID must be consecutive of a table since we allow user removing a property from a table now.

src/include/parser/ddl/drop_column.h

src/include/binder/bind/bound_drop_column.h

src/include/common/statement_type.h

src/binder/expression_binder.cpp

src/storage/store/rel_table.cpp

src/storage/wal_replayer.cpp

src/storage/wal_replayer_utils.cpp

acquamarin requested a review from andyfengHKU January 11, 2023 05:55

andyfengHKU approved these changes Jan 11, 2023

View reviewed changes

acquamarin force-pushed the drop-col branch from 123995c to 47d37df Compare January 11, 2023 08:04

add support for deleting property

884e930

acquamarin force-pushed the drop-col branch from 47d37df to 884e930 Compare January 11, 2023 08:21

acquamarin merged commit d206fa8 into master Jan 11, 2023

acquamarin deleted the drop-col branch January 11, 2023 08:36

ray6080 changed the title ~~add support for deleting property~~ Delete node/rel properties without transactions Jan 29, 2023

ray6080 added the feature New features or missing components of existing features label Jan 29, 2023

Provide feedback