Jarvx200 commented on PR #764: URL: https://github.com/apache/incubator-graphar/pull/764#issuecomment-3339286608
Hey there, thanks a lot for the feedback. Looking back it seems a bit too complicated and it might be harder to mentain. I will try to find a middleground so I respect DRY principles but don't overcomplicate the codebase. Maybe embedding a small builder as an instance variable for common fields could be a bit of a better idea? -- 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]
