Remove no-op OsPrint methods.

They have the same no-op implementation as the overridden base method,
and they aren't called anywhere.

src/backend/gporca/libgpopt/include/gpopt/base/CDrvdPropCtxtRelational.h

@@ -61,12 +61,6 @@ public:
 	// dtor
 	~CDrvdPropCtxtRelational() override = default;
 
-	// print
-	IOstream &
-	OsPrint(IOstream &os) const override
-	{
-		return os;
-	}
 
 #ifdef GPOS_DEBUG
 

src/backend/gporca/libgpopt/include/gpopt/base/COptCtxt.h

@@ -283,10 +283,6 @@ public:
 	// return true if all enforcers are enabled
 	static BOOL FAllEnforcersEnabled();
 
-#ifdef GPOS_DEBUG
-	IOstream &OsPrint(IOstream &) const override;
-#endif	// GPOS_DEBUG
-
 };	// class COptCtxt
 }  // namespace gpopt
 

src/backend/gporca/libgpopt/src/base/COptCtxt.cpp

@@ -147,24 +147,3 @@ COptCtxt::FAllEnforcersEnabled()
 
 	return !fEnforcerDisabled;
 }
-
-
-#ifdef GPOS_DEBUG
-//---------------------------------------------------------------------------
-//	@function:
-//		COptCtxt::OsPrint
-//
-//	@doc:
-//		debug print -- necessary to override abstract function in base class
-//
-//---------------------------------------------------------------------------
-IOstream &
-COptCtxt::OsPrint(IOstream &os) const
-{
-	// NOOP
-	return os;
-}
-
-#endif	// GPOS_DEBUG
-
-// EOF

src/backend/gporca/libgpos/include/gpos/common/CDynamicPtrArray.h

@@ -396,13 +396,6 @@ public:
 		return result;
 	}
 
-	IOstream &
-	OsPrint(IOstream &os) const override
-	{
-		// do nothing, for now
-		return os;
-	}
-
 };	// class CDynamicPtrArray
 }  // namespace gpos