Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21403
@maryannxue as I said my initial proposal was like that. I think that this
has the advantage of avoiding some code duplication as the same logic which is
added in ResolveInValues has to be spread over all the places where a In is
build and avoiding to change the In signature, so that if a user is using In
directly in his/her code we don't break it. On the other side, I agree with you
that the approach having a `Seq[Expression]` is cleaner IMO (that's why it was
my original proposal). @cloud-fan @gatorsmile what do you think about this?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
