emkornfield commented on pull request #12216: URL: https://github.com/apache/arrow/pull/12216#issuecomment-1022894186
OK I found one bug. We don't handle the case when structs have [repeated parents](https://github.com/apache/arrow/blob/master/cpp/src/parquet/arrow/reader.cc#L757). I thought I had handled this case, but maybe a commit got dropped. I would have expected [this test](https://github.com/apache/arrow/blob/master/cpp/src/parquet/arrow/reconstruct_internal_test.cc#L1141) to have caught it. I wonder i re-arranging the data might have caught it or this is just a red-herring. Let me know if this makes sense and if you want to try to reproduce/fix the issue otherwise I can try to do it. -- 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
