mladjan-gadzic commented on PR #4443:
URL: https://github.com/apache/ozone/pull/4443#issuecomment-1495699588

   > It would be ideal to reuse the State Enum type. Would the answer in this 
stackoverflow article help address the import problem? 
https://stackoverflow.com/questions/66524793/how-to-use-the-enum-of-one-proto-file-in-another-file
   
   I like that idea as well. I've tried it, but there is an issue with backward 
compatibility: `[ERROR] Failed to execute goal 
com.salesforce.servicelibs:proto-backwards-compatibility:1.0.7:backwards-compatibility-check
 (default) on project hdds-interface-server: Backwards compatibility check 
failed! You can override this by specifying allowBreakingChanges=true -> [Help 
1]`. I'd say we should not break it, especially because I am not aware of the 
consequences it could have on the system


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to