Uploaded image for project: 'Hadoop Distributed Data Store'
  1. Hadoop Distributed Data Store
  2. HDDS-378

Remove dependencies between hdds/ozone and hdfs proto files

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.3.0, 0.4.0
    • Component/s: None
    • Labels:
    • Target Version/s:

      Description

      It would be great to make the hdds/ozone proto files independent from hdfs proto files. It would help as to start ozone with multiple version of hadoop version.

      Also helps to make artifacts from the hdds protos: HDDS-220

      Currently we have a few unused "hdfs.proto" import in the proto files and we use the StorageTypeProto from hdfs:

      cd hadoop-hdds
      grep -r "hdfs" --include="*.proto"
      common/src/main/proto/ScmBlockLocationProtocol.proto:import "hdfs.proto";
      common/src/main/proto/StorageContainerLocationProtocol.proto:import "hdfs.proto";
      
       cd ../hadoop-ozone
      grep -r "hdfs" --include="*.proto"
      common/src/main/proto/OzoneManagerProtocol.proto:import "hdfs.proto";
      common/src/main/proto/OzoneManagerProtocol.proto:    required hadoop.hdfs.StorageTypeProto storageType = 5 [default = DISK];
      common/src/main/proto/OzoneManagerProtocol.proto:    optional hadoop.hdfs.StorageTypeProto storageType = 6;
      

      I propose to

      1.) remove the hdfs import statements from the proto files
      2.) Copy the StorageTypeProto and create a Hdds version from it (without PROVIDED)

        Attachments

        1. HDDS-378.001.patch
          45 kB
          Marton Elek
        2. HDDS-378.002.patch
          84 kB
          Marton Elek
        3. HDDS-378.003.patch
          26 kB
          Marton Elek

          Issue Links

            Activity

              People

              • Assignee:
                elek Marton Elek
                Reporter:
                elek Marton Elek
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: