carols10cents opened a new pull request #8515: URL: https://github.com/apache/arrow/pull/8515
Hey @nevi-me, I was looking into helping out with the `def_levels` and `rep_levels` handling in `get_levels` in arrow_writer.rs, and the logic is... quite complex! I honestly have no idea how you're planning to get it to be the same as the C++ code without a direct port of the C++ algorithm; the code looks completely different right now so I feel like I have no chance of helping to fix the Rust code by looking at the C++ code. I feel like you've mentioned that you're working on fixing the def/rep level stuff somewhere and that it was taking a while; what do you think of a more direct correspondence with the C++ code that I've started here? Are there reasons not to go this way? I wanted to see if this kind of direction would be helpful in getting the rust-parquet-arrow-writer branch merged into master, or if this is too much or too little or the wrong direction. Please advise, thank you! ---------------------------------------------------------------- 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: [email protected]
