Github user maryannxue commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21875#discussion_r205266067
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
 ---
    @@ -172,7 +172,11 @@ private[sql] case class JDBCRelation(
     
       // Check if JDBCRDD.compileFilter can accept input filters
       override def unhandledFilters(filters: Array[Filter]): Array[Filter] = {
    -    filters.filter(JDBCRDD.compileFilter(_, 
JdbcDialects.get(jdbcOptions.url)).isEmpty)
    +    if (jdbcOptions.pushDownPredicate) {
    --- End diff --
    
    Yes, this is the only source of truth for defining handled/unhandled. The 
caller calls this method and push "handled" to scanTable, in this case JDBCRDD.


---

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

Reply via email to