SemyonSinchenko commented on issue #738: URL: https://github.com/apache/incubator-graphar/issues/738#issuecomment-3245159980
From my side: 1. YAML-parser (doesn't matter, will it be a `Jackson-YAML` or `SnakeYAML`) dependency should be shaded imo. 2. The main `info` classes should be made abstract and require from the user to provide an implementation of `String readYAML(String pathString)` method. The main usage of GAR is cloud and distributed FS, so, we should not depend on Hadoop, Ozone Java Client, S3 or anything like this. We may have a LocalFS implementation as an example and for testing, but for real system users / developers should provide implementation for reading YAML-files. @yangxk1 @Thespica Thoughts? -- 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]
