[GitHub] [drill] luocooong merged pull request #2286: [DOC UPDATE] Translate Tutorial to Chinese 7/14

2021-08-11 Thread GitBox
luocooong merged pull request #2286: URL: https://github.com/apache/drill/pull/2286 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr..

Re: Strange query crash

2021-08-11 Thread luoc
Hello Ted, I think the error stack looks can be deceiving. There is a ticket (DRILL-4254 ) related to the issue. I recommend that you upgrade to latest version if is cause by the schema change. > 2021年8月11日 上午4:18,Ted Dunning 写道: > > > I am

[GitHub] [drill] Leon-WTF commented on a change in pull request #2290: DRILL-7984: Support clickhouse by JDBC plugin

2021-08-11 Thread GitBox
Leon-WTF commented on a change in pull request #2290: URL: https://github.com/apache/drill/pull/2290#discussion_r686905466 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/clickhouse/ClickhouseDialect.java ## @@ -0,0 +1,73 @@ +/* + * Licensed t

[GitHub] [drill] Leon-WTF commented on a change in pull request #2290: DRILL-7984: Support clickhouse by JDBC plugin

2021-08-11 Thread GitBox
Leon-WTF commented on a change in pull request #2290: URL: https://github.com/apache/drill/pull/2290#discussion_r686905466 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/clickhouse/ClickhouseDialect.java ## @@ -0,0 +1,73 @@ +/* + * Licensed t

Re: Strange query crash

2021-08-11 Thread Ted Dunning
Interesting. I will see what I can do about the upgrade. On Wed, Aug 11, 2021 at 7:16 AM luoc wrote: > Hello Ted, > I think the error stack looks can be deceiving. There is a ticket > (DRILL-4254 ) related > to the issue. I recommend that yo

[GitHub] [drill] Leon-WTF commented on a change in pull request #2290: DRILL-7984: Support clickhouse by JDBC plugin

2021-08-11 Thread GitBox
Leon-WTF commented on a change in pull request #2290: URL: https://github.com/apache/drill/pull/2290#discussion_r686905466 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/clickhouse/ClickhouseDialect.java ## @@ -0,0 +1,73 @@ +/* + * Licensed t

[GitHub] [drill] vvysotskyi commented on pull request #2289: DRILL-7985: Support Mongo aggregate, union, project, limit, sort pushdowns

2021-08-11 Thread GitBox
vvysotskyi commented on pull request #2289: URL: https://github.com/apache/drill/pull/2289#issuecomment-897030162 Hi @julianhyde, part of this pull request introduces an API for adding support of pushdowns to the storage plugins, and it is something similar to what Calcite adapters do. Do

[GitHub] [drill] julianhyde commented on pull request #2289: DRILL-7985: Support Mongo aggregate, union, project, limit, sort pushdowns

2021-08-11 Thread GitBox
julianhyde commented on pull request #2289: URL: https://github.com/apache/drill/pull/2289#issuecomment-897089942 @vvysotskyi Of course it depends what is feasible for Drill, given its current APIs, users, and code base. But I introduced Calcite's model of adapters and am still a big

[GitHub] [drill] cgivre commented on pull request #2289: DRILL-7985: Support Mongo aggregate, union, project, limit, sort pushdowns

2021-08-11 Thread GitBox
cgivre commented on pull request #2289: URL: https://github.com/apache/drill/pull/2289#issuecomment-897109218 > @vvysotskyi Of course it depends what is feasible for Drill, given its current APIs, users, and code base. > > But I introduced Calcite's model of adapters and am still a b

[GitHub] [drill] julianhyde commented on pull request #2289: DRILL-7985: Support Mongo aggregate, union, project, limit, sort pushdowns

2021-08-11 Thread GitBox
julianhyde commented on pull request #2289: URL: https://github.com/apache/drill/pull/2289#issuecomment-897230616 Note that Calcite's "adapter" abstraction isn't as strong as Drill's. It's just a bunch of rules (and often a calling convention). So maybe you can just pull in the rules you n