jinyius
jinyius
thanks for the review. updated to handle the logging perf concern as well as fixing the javadoc errors.
> Mostly looks reasonable, I'm not too familiar with parquet-mr @shangxinli can you recommend someone who might be able to give a better review? pinging @shangxinli :)
> I would also like to recommend adding @matthieun as a co-author to this PR, per the discussion in the parallel PR. ~how do you do this?~ i think i...
can someone retry the github actions? there seemed to have been a transient issue that caused one of the test/build targets to fail. i'd like to get this change in...
@ggershinsky i'd love to just hit the button. i don't see it. the workflow for travis ci had a failure due to a transient connection issue, and so it wasn't...
yeah, i still don't see a button to merge. it now shows everything approved, checks passed, and no conflicts. i think a committer needs to merge.