robertmu opened a new pull request, #31: URL: https://github.com/apache/cloudberry-gpbackup/pull/31
This commit modifies the operator metadata integration test to support Cloudberry in addition to the original Greenplum support. The original test created a unary prefix operator, which worked fine for Greenplum. However, Cloudberry (based on PostgreSQL 14) no longer supports user-defined unary operators, causing the test to fail. To ensure compatibility across both platforms, the test now uses a binary operator approach universally, which is supported by both Greenplum and Cloudberry. This eliminates the need for conditional logic while maintaining the same test coverage. Changes: - Replace unary prefix operator with binary operator - Use `binary_op_func(bigint, bigint)` instead of `numeric_fac` - Update operator syntax to `(LEFTARG = bigint, RIGHTARG = bigint)` - Remove database-specific conditional branching This change ensures the test passes on both Greenplum and Cloudberry without compromising the metadata validation logic. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org For additional commands, e-mail: commits-h...@cloudberry.apache.org
