Github user cloud-fan commented on the issue:

    https://github.com/apache/spark/pull/21106
  
    I think `UnsafeProjectionCreator` is only useful for tests, because in 
production we should always try codegen first, and fallback to interpretation 
if codegen fails. There is no need to disable the fallback(always codegen) or 
skip the codegen(always interpretation) in production.
    
    How about we provide a sql conf for test only, to be able to disable the 
fallback(always codegen) or skip the codegen(always interpretation)?


---

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

Reply via email to