pitrou commented on code in PR #45818: URL: https://github.com/apache/arrow/pull/45818#discussion_r2102076201
########## python/pyarrow/scalar.pxi: ########## @@ -238,6 +241,9 @@ cdef class UInt8Scalar(Scalar): cdef CUInt8Scalar* sp = <CUInt8Scalar*> self.wrapped.get() return sp.value if sp.is_valid else None + def __int__(self): + return self.as_py() Review Comment: I think that `__index__` is sufficient in all cases, since it will be used if `__int__` doesn't exist (the reverse is not true). -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org