parquet-java icon indicating copy to clipboard operation
parquet-java copied to clipboard

parquet-hadoop tests to work with hadoop 3.4.0

Open asfimport opened this issue 1 year ago • 2 comments

Test runs of parquet-hadoop with -Dhadoop.version=3.4.0 fail because there's a logback jar on the classpath, which screws things up (mostly seemingly because it suddenly logs at debug)

HADOOP-19084 should have fixed this, as it explicitly excludes the export. clearly not. created HADOOP-19153 as a followup.

For now, all parquet needs is to exclude logback as a hadoop-common dependency. I'll be extra cautious and get rid of all log dependencies.

Reporter: Steve Loughran / @steveloughran

Related issues:

PRs and other links:

Note: This issue was originally created as PARQUET-2459. Please see the migration documentation for further details.

asfimport avatar Apr 17 '24 15:04 asfimport

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.42%. Comparing base (bf28589) to head (7bf70de). Report is 121 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4107      +/-   ##
==========================================
- Coverage   38.44%   38.42%   -0.03%     
==========================================
  Files         723      724       +1     
  Lines       33247    33274      +27     
  Branches     4889     4725     -164     
==========================================
+ Hits        12783    12785       +2     
- Misses      19315    19865     +550     
+ Partials     1149      624     -525     
Flag Coverage Δ
api-python 90.85% <ø> (ø)
catalog 11.43% <ø> (-0.01%) :arrow_down:
lambda 87.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 17 '24 03:08 codecov[bot]

Closing old @akarve PRs

drernie avatar Dec 24 '24 18:12 drernie