vibhatha commented on a change in pull request #11993:
URL: https://github.com/apache/arrow/pull/11993#discussion_r774623561



##########
File path: python/pyarrow/array.pxi
##########
@@ -988,12 +988,64 @@ cdef class Array(_PandasConvertible):
         """
         Total number of bytes consumed by the elements of the array.
         """
+        return self.get_refererenced_buffer_size()
+
+    def get_total_size_by_buffer(self):

Review comment:
       Good catch. 
   
   This is a question I also have. Here the Python method counts the elements 
as same as it is done in the C++ method `TotalBufferSize`. For now I kept it as 
this is still WIP. I also felt like we must remove the calculation done in 
Python and just use the C++ method. Am I following this right? 




-- 
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]


Reply via email to