You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "mapleFU (via GitHub)" <gi...@apache.org> on 2023/02/01 06:20:38 UTC

[GitHub] [arrow] mapleFU commented on pull request #33897: GH-33652: [C++][Parquet] Add interface total_compressed_bytes_written

mapleFU commented on PR #33897:
URL: https://github.com/apache/arrow/pull/33897#issuecomment-1411526998

   > Yes, I think adding the comment should help understand. I wish these parts of the codebase were better documented.
   > 
   > Are you still planning on adding tests?
   
   Yes, I will, but I'm a little busy this week, and adding tests for bytes_written is boring and troublesome, because we only know that I "would no less than xx bytes" or "should less than xx bytes", but actually don't know it's really length when writing...
   
   I'll add it this weekend...


-- 
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-unsubscribe@arrow.apache.org

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