citus node deep copying codes should use copyObject functions
Open
1-2 days
technical debt
- Dominant language
- C
- Stars
- 12.8k
- Forks
- 794
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 31
Description
We introduced copy functions for all citus nodes. There are some places in the code that allocated memory using palloc0 for citus nodes and use their own copy functions.
```c
master_metadata_utility.c::CopyShardInterval()
master_metadata_utility.c::CopyShardPlacement()
```
are some examples.
We should find out other places in the code that does the similar thing and make them use copyObject() function instead.
Contributor guide
Assessment
This issue has not been assessed yet.