wilfred-s commented on pull request #22: URL: https://github.com/apache/incubator-yunikorn-scheduler-interface/pull/22#issuecomment-666989624
We have had a long offline discussion about this and we have a problem with the way the interface was designed. That problem cannot be fixed as part of this change. We need to work on that outside of this change. The problem with the way this is designed is that whenever we do anything we need to wrap it in an `UpdateRequest` or `UpdateResponse` message as that is the only message the interface allows us to send between the core and the shim. I am filing a follow up jira to discuss and modify how this was designed and implemented. Having a simple contract with just one message pushes us into a complicated message handling and creation on both sides of the contract I'll pull in my request for changes and approve the addition to the message. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
