robertmu opened a new pull request, #33: URL: https://github.com/apache/cloudberry-gpbackup/pull/33
This commit addresses a test failure in the `GetOperatorClasses` integration tests, which was observed on newer database versions like Cloudberry (based on PG14). The investigation revealed an inconsistency in how Greenplum 7 (based on PG12) and Cloudberry handle the `storagetype` for operator classes. When an operator class's `STORAGE` type is the same as its `FOR TYPE` and `opckeytype` is not 0: - In GPDB7, the expression `opckeytype::regtype` evaluates to "-". - In CBDB, the same expression evaluates to the actual type name (e.g., "integer"). This change introduces a `CASE` statement to explicitly check if `opcintype` equals `opckeytype`. This normalizes the behavior across both database versions, ensuring that `storagetype` is consistently represented as "-" when the storage type is the same as the input type, which allows the failing test to pass. -- 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
