On 27.05.25 05:06, Quan Zongliang wrote: > I noticed that the subtype of AlterDomainStmt is directly using > constants in the code. It is not conducive to the maintenance and > reading of the code. Based on the definition of AlterTableType, use > "AD_" as the prefix. Define several macros to replace the original > characters. > The subtype of AlterTableCmd is defined using an enumeration. The > subtypes of AlterDomainStmt are relatively few in number, and the > original definition uses characters. These definitions still use > characters and maintain the values unchanged. If some plugins or tools > are also processing AlterDomainStmt, there will be no errors.
You can still make it an enum and assign the currently in use values to the new symbols, like
