thelabdude commented on pull request #181:
URL: https://github.com/apache/solr/pull/181#issuecomment-865196735


   > I think this conflicts with #177 can you and Mark chat about the proper 
implementation of `toSolrLiteral`/`translateTerms`
   
   @markrmiller I think this PR will get merged before your upgrade ... once 
you merge in my changes here, you can drop the `translateTerms` call and just 
use my changes as I switched to calling `getValue2()` as you're doing in your 
PR ... thanks for catching this @madrob ;-)


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to