This is an automated email from the ASF dual-hosted git repository.

maxyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudberry.git

commit b3fdca2567e193c7185782a46951a03ca7ac5432
Author: Chris Hajas <[email protected]>
AuthorDate: Mon Nov 27 17:03:44 2023 -0800

    Fix DbgStr when printing DP structs in Orca
    
    Since CJoinOrderDP and CJoinOrderDPv2 inherit from CJoinOrder, we had
    multiple definitions  and DbgStr  didn't work.
---
 src/backend/gporca/libgpopt/include/gpopt/xforms/CJoinOrder.h | 2 +-
 src/backend/gporca/libgpopt/src/xforms/CJoinOrder.cpp         | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/backend/gporca/libgpopt/include/gpopt/xforms/CJoinOrder.h 
b/src/backend/gporca/libgpopt/include/gpopt/xforms/CJoinOrder.h
index 44d98214f9..6dbbcdf4d1 100644
--- a/src/backend/gporca/libgpopt/include/gpopt/xforms/CJoinOrder.h
+++ b/src/backend/gporca/libgpopt/include/gpopt/xforms/CJoinOrder.h
@@ -34,7 +34,7 @@ using namespace gpos;
 //             Helper class for creating compact join orders
 //
 //---------------------------------------------------------------------------
-class CJoinOrder : public DbgPrintMixin<CJoinOrder>
+class CJoinOrder
 {
 public:
        enum EPosition
diff --git a/src/backend/gporca/libgpopt/src/xforms/CJoinOrder.cpp 
b/src/backend/gporca/libgpopt/src/xforms/CJoinOrder.cpp
index be00659366..0c21563024 100644
--- a/src/backend/gporca/libgpopt/src/xforms/CJoinOrder.cpp
+++ b/src/backend/gporca/libgpopt/src/xforms/CJoinOrder.cpp
@@ -27,7 +27,6 @@
 
 using namespace gpopt;
 
-FORCE_GENERATE_DBGSTR(CJoinOrder);
 FORCE_GENERATE_DBGSTR(CJoinOrder::SEdge);
 FORCE_GENERATE_DBGSTR(CJoinOrder::SComponent);
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to