kiszk commented on issue #6981:
URL: https://github.com/apache/arrow/pull/6981#issuecomment-616759329


   Thank you for staring the discussion. I will watch at the thread.
   
   Yeah, `LETypedBufferBuilder` makes sense. It looks better than adding 
`AppendLE`.
   
   Regarding `Serialize`, it looks a good place where the class has both types 
of an element for Arrow and Parquet. But, encoding (i.e RLE and others) happens 
in `encoding.cc`. Let me check it tomorrow.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to