noblepaul edited a comment on issue #1124: SOLR-14151 :Schema components to be loadable from packages URL: https://github.com/apache/lucene-solr/pull/1124#issuecomment-569546583 >It's as if you can't bear to say you like SRL so you create something else in its image but without the dead weight we agree shouldn't be in SRL. > the SRL you don't like is still there. SRL is fine today as a class & I don't hate it at all. Did I create something else in its image? NO. I just applied some lipstick on to that cute little piglet. It's just an interface. I don't even know why we are discussing SRL here. It's unproductive and we are not solving any problems with that. > A down side to this approach is that the system is now more complex as a whole; Yes, the system is more complex because it is expected to do new things. We just hide the complexity behind abstractions. People do not have to deal with the complexity unless they need to. They should be given clear interfaces/APIs to deal with it. If they ever need to know how it works, they just peel off the interface and look at the implementation. Now, we have a solution that works. This already covers >90% of what most users would need. We will have more productive debate if we start discussing - Is there a problem with the PR? Is it bad code? Is it working wrong? Is it inefficient? - How do we cover the last mile? If 'n' developers are asked to solve a problem, we will get 'n' solutions. We can keep on bikeshedding why solution A is not as good as solution B or vice versa. We just solve it and move on. Solr has a lot of problems to solve. We are just worried about the color of the bike shed
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
