@vvysotskyi thanks for your comments. Changes are made as you suggested. As you mentioned early, some tests are made to test this functionality: Both jdk and janio compiler works fine. Constants with different types(string, double, long, decimal, time, timestamp and so on) are tested. But some tests beside the Travis are not passed, a lot of `No default schema selected` errors occured, it seems that these errors are not connected to this change.
[ Full content available at: https://github.com/apache/drill/pull/1481 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org
