Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/1170
Over the last year, we've tended to favor including unit tests with each
PR. There don't seem to be any with this one, yet we are proposing to make a
fairly complex change. Perhaps tests can be added.
Further, by having good tests, we don't have to debate how Drill will
handle the scenarios discussed in an earlier comment: we just code 'em up and
try 'em out, letting Drill speak for itself. We can then decide whether or not
we like the results, rather than discussing hypotheticals.
---
