Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19726
  
    Oh, I had to be clear, I think Sphinx 1.6.6 is in development and upgrading 
it is actually a bigger issue I think.
    
    I tried other ways I could think ahead actually, but failed to figure out a 
better way. So, I meant LGTM assuming we should generally avoid this workaround 
but it's one exception (or possibly few in the future), and it is by a bug 
existing in Sphinx 1.6.6 <.
    
    But let's wait for more feedback just in case I missed something and 
someone has a better idea.


---

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

Reply via email to