gerlowskija commented on PR #4738: URL: https://github.com/apache/solr/pull/4738#issuecomment-5362268002
On the Community Meetup I expressed some worries about the maintenance of this code. It feels like a maintenance risk to merge 3k LOC that's largely LLM-generated, in a testing framework (i.e. Selenium) that few or none of us are that familiar with, and that covers code that few or none of us are familiar with. Practically - my primary concern is/was that the tests might be flaky and block development in other areas, or provide friction for potential contributors. That said, I think in this case it's more than fair to "cross that bridge when we come to it". I don't want to hold work up based on concerns (i.e. flakiness) that may never come to pass. That's all to say - don't let my concerns voiced on the Meetup slow this down at all. I trust you guys to circle back if we find this does cause issues down the line 👍 -- 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]
