paypal / NNAnalytics

NameNodeAnalytics is a self-help utility for scouting and maintaining the namespace of an HDFS instance.
Apache License 2.0
110 stars 71 forks source link

Add support for HDFS 2.10.0 #300

Closed shvachko closed 3 years ago

shvachko commented 3 years ago
shvachko commented 3 years ago

Guess need to update supported versions in the doc.

pjeli commented 3 years ago

Hey Konstantin. Sorry it has been a while. Can you please re-base to the latest commit on master branch and push to your PR branch again? It turns out Travis CI (in the way I was using it) is deprecated now and so no CI was happening for NNA.

I have merged in the commons-io fix as that was affecting the entire build, so your new PR should be simpler.

Again, sorry, but at least next PR push will have CI for build.

pjeli commented 3 years ago

One more thing will need to be updated, should be done here as part of this issue:

shvachko commented 3 years ago

Rebased the patch. Updated gradle.yml

codecov[bot] commented 3 years ago

Codecov Report

Merging #300 (e43bf89) into master (75e04d2) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #300   +/-   ##
=========================================
  Coverage     74.39%   74.39%           
  Complexity     1067     1067           
=========================================
  Files            55       55           
  Lines          7023     7023           
  Branches        631      631           
=========================================
  Hits           5225     5225           
  Misses         1410     1410           
  Partials        388      388           
Impacted Files Coverage Δ Complexity Δ
...he/hadoop/hdfs/server/namenode/NameNodeLoader.java 74.40% <100.00%> (ø) 46.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 75e04d2...e43bf89. Read the comment docs.

pjeli commented 3 years ago

Thanks for the contribution @shvachko !