[GitHub] [calcite] wojustme removed a comment on pull request #2702: [CALCITE-4800] Define a new rule of SortToSortUnifyRule in SubstitutionVisitor to support sort's match

2022-01-28 Thread GitBox
wojustme removed a comment on pull request #2702: URL: https://github.com/apache/calcite/pull/2702#issuecomment-1024816813 Handle for Sort's mv-match - Equal collation's info, we just check offset and fetch. Query could be rewritten by mv, when mv's data-range must be larger than

[GitHub] [calcite] wojustme opened a new pull request #2702: [CALCITE-4800] Define a new rule of SortToSortUnifyRule in SubstitutionVisitor to support sort's match

2022-01-28 Thread GitBox
wojustme opened a new pull request #2702: URL: https://github.com/apache/calcite/pull/2702 Link to [ISSUE-4800](https://issues.apache.org/jira/browse/CALCITE-4800) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [calcite] wojustme commented on pull request #2702: [CALCITE-4800] Define a new rule of SortToSortUnifyRule in SubstitutionVisitor to support sort's match

2022-01-28 Thread GitBox
wojustme commented on pull request #2702: URL: https://github.com/apache/calcite/pull/2702#issuecomment-1024816813 Handle for Sort's mv-match - Equal collation's info, we just check offset and fetch. Query could be rewritten by mv, when mv's data-range must be larger than query's.

[GitHub] [calcite] wojustme closed pull request #2702: [CALCITE-4800] Define a new rule of SortToSortUnifyRule in SubstitutionVisitor to support sort's match

2022-01-28 Thread GitBox
wojustme closed pull request #2702: URL: https://github.com/apache/calcite/pull/2702 -- 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:

[GitHub] [calcite] beyond1920 commented on pull request #2703: Site: Add a section for putting external resources under the community page

2022-01-28 Thread GitBox
beyond1920 commented on pull request #2703: URL: https://github.com/apache/calcite/pull/2703#issuecomment-1024807468 @zabetak The changes looks good. Thanks a lot. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[calcite-site] branch master updated: Site: Add external resources section in the community page

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-site.git The following commit(s) were added to refs/heads/master by this push: new a6fa17e Site: Add external resources

[calcite] branch site updated: Site: Add external resources section in the community page

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch site in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/site by this push: new fabef05 Site: Add external resources section in

[calcite] branch master updated: Site: Add external resources section in the community page

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new 8b55e31 Site: Add external resources section

[GitHub] [calcite] zabetak commented on pull request #2703: Site: Add a section for putting external resources under the community page

2022-01-28 Thread GitBox
zabetak commented on pull request #2703: URL: https://github.com/apache/calcite/pull/2703#issuecomment-1024736759 I did a few changes (hope you are OK with them) and pushed it. Thanks a lot for the PR @beyond1920 ! -- This is an automated message from the Apache Git Service. To respond

[GitHub] [calcite] zabetak closed pull request #2703: Site: Add a section for putting external resources under the community page

2022-01-28 Thread GitBox
zabetak closed pull request #2703: URL: https://github.com/apache/calcite/pull/2703 -- 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:

[calcite-site] 02/02: Site: Add "calcite-clj - Use Calcite with Clojure" in talks section

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-site.git commit 0df40dc3ae730d2efa0729903d6c4b9bdce4b0c5 Author: Stamatis Zampetakis AuthorDate: Fri Jan 28 23:26:04 2022 +0100

[calcite-site] 01/02: Site: Fix typo in howto.md

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-site.git commit 0e31918c5f1259920f2d7c5c5abf0ab43209aaf2 Author: Stamatis Zampetakis AuthorDate: Fri Jan 28 23:25:15 2022 +0100

[calcite-site] branch master updated (74e4896 -> 0df40dc)

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite-site.git. from 74e4896 Site: Add Alessandro Solimando as committer new 0e31918 Site: Fix typo in howto.md new

[calcite] branch site updated: Site: Add "calcite-clj - Use Calcite with Clojure" in talks section

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch site in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/site by this push: new 48f4bf8 Site: Add "calcite-clj - Use Calcite

[GitHub] [calcite] zabetak commented on pull request #2704: Added calcite-clj references to adpater page and talks

2022-01-28 Thread GitBox
zabetak commented on pull request #2704: URL: https://github.com/apache/calcite/pull/2704#issuecomment-1024691481 Thanks for the PR @ieugen . I left out the changes in the adapter page since we are not talking about an adapter there. Let's put that in the new community section ;) --

[calcite] branch master updated: Site: Add "calcite-clj - Use Calcite with Clojure" in talks section

2022-01-28 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new c80b948 Site: Add "calcite-clj - Use Calcite

[GitHub] [calcite] zabetak closed pull request #2704: Added calcite-clj references to adpater page and talks

2022-01-28 Thread GitBox
zabetak closed pull request #2704: URL: https://github.com/apache/calcite/pull/2704 -- 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:

[GitHub] [calcite] jnturton commented on pull request #2698: [CALCITE-4992] Close the ElasticSearch RestClient in ElasticsearchTransport

2022-01-28 Thread GitBox
jnturton commented on pull request #2698: URL: https://github.com/apache/calcite/pull/2698#issuecomment-1024449415 @ILuffZhe @zabetak, a colleague of mine has raised a concern that recreating the RestClient for every request, as I've done in this iteration, might negatively affect

[GitHub] [calcite] korlov42 commented on pull request #2623: CALCITE-4913 Correlated variables in a select list are not deduplicated

2022-01-28 Thread GitBox
korlov42 commented on pull request #2623: URL: https://github.com/apache/calcite/pull/2623#issuecomment-1024010004 Hi, @zabetak! Seems like 10 days have passed. Now it's my turn to be unavailable for next 10 days, but I really hope you'll find time to review this. -- This is an