murblanc commented on pull request #1730:
URL: https://github.com/apache/lucene-solr/pull/1730#issuecomment-671509879


   
   > The current concrete classes do not use/implement these interfaces. These 
interfaces will only be a part of implementations. for instance, the 
`LazySolrCluster` is one of the impl. In the future we should add a couple more
   
   @noble what is the target use case of the interface and lazy implementation? 
I thought your aim was to create interfaces to existing internal classes 
therefore I expected them to implement these interfaces so these interfaces to 
be used in the code replacing the actual classes...
   Maybe it's just me not understading your intention here. 


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to