Yes, at this moment we can detect that it's explicit cross join because here inspected object is SqlNode, but later after conversion to RelNode this `JoinType.CROSS` will be lost, instead we only have `JoinRelType.INNER` in the node. So to preserve original type additional tricks with query context are necessary. What you suggest is actually **Option 2** from my comment in [DRILL-786](https://issues.apache.org/jira/browse/DRILL-786) , but as I remember we agreed to go with **Option 3** . So what are the reasons to move back and use Option 2 ?
[ Full content available at: https://github.com/apache/drill/pull/1488 ] This message was relayed via gitbox.apache.org for [email protected]
