babelfish-for-postgresql / babelfish_extensions / 13816463706

77%

push

github

alter procedure/function will unexpectly delete pg_depend records (#3554)


Backgroud:
Previously when we implement alter procedure/function feature, we did
follow steps in update metadata:
1. Create a new procedure/function
2. use the old oid to replace the new oid in the pg_proc metadata
3. clean the pg_depend records refer to the new oid

Analysis:
But for the first step in creating a new proc/function, if the parameter
list is not change, it'll just update the old tuple, so in such case
old oid will be the same with new oid, in such case we should not drop
pg_depend records.

Fix:
Skip drop pg_depend records if old oid = new oid

Issue-resolved: BABEL-5601
Signed-off-by: Zhibai Song <szh@amazon.com>

11 of 11 new or added lines in 2 files covered. (100.0%)

47356 of 63122 relevant lines covered (75.02%)

188599.37 hits per line