Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21388
@hvanhovell
I also think someone might not want to have reflection magic (I was the one
but realized I should do it), so I'm happy to close the PR when others voice
same opinion on this too.
For me, reflection looks like only way to achieve `Can we automate these
'pass through' operations?`, so if we decide to reject the approach, we might
be better to either remove the line, or add description on restriction(s)
instead, unless we have another immediate idea to achieve it without reflection.
Btw, I'd be very happy if you are happy to spend some time to explain which
points make you being concerned about reflection in planner. Maybe adding the
description explicitly would avoid the similar trial on contributors and save
our time.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
