pitrou commented on code in PR #13364:
URL: https://github.com/apache/arrow/pull/13364#discussion_r897146358
##########
cpp/src/arrow/type.h:
##########
@@ -126,7 +126,8 @@ struct ARROW_EXPORT DataTypeLayout {
///
/// Simple datatypes may be entirely described by their Type::type id, but
/// complex datatypes are usually parametric.
-class ARROW_EXPORT DataType : public detail::Fingerprintable {
+class ARROW_EXPORT DataType : public std::enable_shared_from_this<DataType>,
Review Comment:
Using `enable_shared_from_this` sounds fine to me. There is basically no use
case for creating non-shared-ptr'ed DataTypes anyway.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
