Remove unused method alluxio.hadoop.HadoopUtils#toStringHadoopInputSplit

Alluxio / Community

New Contributor Tasks for Alluxio

20 stars 38 forks source link

Remove unused method alluxio.hadoop.HadoopUtils#toStringHadoopInputSplit #602

Closed apc999 closed 3 years ago

apc999 commented 3 years ago

This method https://github.com/Alluxio/alluxio/blob/a98c6dbe312a125ee09daf4fa08c7498cd67e951/core/client/hdfs/src/main/java/alluxio/hadoop/HadoopUtils.java#L135 is unused across the codebase.

We can remove it

dbw9580 commented 3 years ago

There're more methods in this class HadoopUtils that my IDE suggests are not used, e.g. getAlluxioFileName and toStringHadoopFileSplit. Should we remove them together, too?

AgastyaSingh3292 commented 3 years ago

Can I work on it?

apc999 commented 3 years ago

sure, go ahead Agastya

On Sat, May 29, 2021 at 6:51 AM Agastya Singh @.***> wrote:

Can I work on it?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Alluxio/new-contributor-tasks/issues/602#issuecomment-850836951, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKZE5HV2I4PTPDHIXNBKBDTQDWNZANCNFSM45KBLAYA .