Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-10548

Remove the long deprecated BlockReaderRemote

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0-alpha1
    • Component/s: hdfs
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      This removes the configuration property {{dfs.client.use.legacy.blockreader}}, since the legacy remote block reader class has been removed from the codebase.

      Description

      To lessen the maintain burden like raised in HDFS-8901, suggest we remove BlockReaderRemote class that's deprecated very long time ago.

      From BlockReaderRemote header:

      • @deprecated this is an old implementation that is being left around
      • in case any issues spring up with the new
        Unknown macro: {@link BlockReaderRemote2}
      • implementation.
      • It will be removed in the next release.

      From BlockReaderRemote2 class header:

      • This is a new implementation introduced in Hadoop 0.23 which
      • is more efficient and simpler than the older BlockReader
      • implementation. It should be renamed to BlockReaderRemote
      • once we are confident in it.

      So even further, after getting rid of the old class, we could rename as the comment suggested: BlockReaderRemote2 => BlockReaderRemote.

        Attachments

        1. HDFS-10548-v1.patch
          25 kB
          Kai Zheng
        2. HDFS-10548-v2.patch
          62 kB
          Kai Zheng
        3. HDFS-10548-v3.patch
          62 kB
          Kai Zheng

          Activity

            People

            • Assignee:
              drankye Kai Zheng
              Reporter:
              drankye Kai Zheng
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: