[
https://issues.apache.org/jira/browse/HDDS-378?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Arpit Agarwal updated HDDS-378:
-------------------------------
Resolution: Fixed
Fix Version/s: 0.4.0
0.3.0
Status: Resolved (was: Patch Available)
+1 I've committed this for now to make progress with building HDDS separately
from Hadoop. We can fix the duplicated enum separately.
Thanks for reporting and fixing this [~elek].
> Remove dependencies between hdds/ozone and hdfs proto files
> -----------------------------------------------------------
>
> Key: HDDS-378
> URL: https://issues.apache.org/jira/browse/HDDS-378
> Project: Hadoop Distributed Data Store
> Issue Type: Improvement
> Reporter: Elek, Marton
> Assignee: Elek, Marton
> Priority: Major
> Labels: newbie
> Fix For: 0.3.0, 0.4.0
>
> Attachments: HDDS-378.001.patch, HDDS-378.002.patch,
> HDDS-378.003.patch
>
>
> 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:
> {code}
> 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;
> {code}
> 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)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
