cloud-fan commented on PR #39524:
URL: https://github.com/apache/spark/pull/39524#issuecomment-1381848426

   @huaxingao I think the v1 filter to v2 translation has a bug
   ```
   case class IsNotNull(attribute: String) extends Filter {
     override def references: Array[String] = Array(attribute)
     override def toV2: Predicate = new Predicate("IS_NOT_NULL", 
Array(FieldReference(attribute)))
   }
   ```
   
   The `attribute` can either be the parsed name (no quoting) if nested filter 
is disabled, or a SQL name otherwise. But `FieldReference.apply` only accepts 
SQL name.


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

To unsubscribe, e-mail: [email protected]

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