On 02.01.26 09:48, Ashutosh Bapat wrote:
#1 [Code] src/backend/commands/propgraphcmds.c:1632 FIXME: Missing index for plppropid column causes sequential scan. Decision needed: (a) add index, or (c) allow seq scan for rare path.The path is rare enough that I think we can allow the seq scan. Given that Peter has marked it as FIXME, it seems we will keep it as is for now, but will revisit if performance becomes an issue. Peter, please correct me if I'm wrong.
Yeah, doesn't seem critical. Maybe "downgrade" to "XXX".
#5 [Doc] doc/src/sgml/queries.sgml:2929,2931 TODO comments for unimplemented features without clear limitation notes.Those TODOs are not visible to users. I think they serve as placeholders to add the documentation when the features are implemented. We may want to remove them, but I see no harm in keeping them for now. Peter, what do you think?
We should keep them and fill them in. :)
