srijeet0406 commented on pull request #5974: URL: https://github.com/apache/trafficcontrol/pull/5974#issuecomment-876539328
I agree that code duplication should be minimized, but as Rawlin specified, this was never meant to be the long term solution, so holding up this last PR(and the project and the 6.0 release) for a major refactor isn't super beneficial IMO. I'm also trying to keep the CDN locks simple without adding a bunch more database tables and such, in accordance with the blueprint. Additionally, while speaking with Dave and the rest of the team, this was supposed to be the "first pass" at CDN locks. We can keep improving it in the next iterations (until it is replaced) and I'd be happy to add a ticket to take care of this in a future HIP sprint. Does that sound good to everyone? -- 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]
