Github user HanumathRao commented on the issue: https://github.com/apache/drill/pull/1117 @ilooner Changes look fine to me. However, as discussed offline I couldn't reproduce the plan which was sorting one of the inputs for a HASHJOIN. This might be because in the hashjoinprule we are not asking for those traits in the first place. However, I do agree that this fix can be a safegaurd in case if that plan happens to show up in some scenarios. Thanks for the fix.
---