[GitHub] [calcite] jinxing64 opened a new pull request #1452: [CALCITE-3338] Error with executeBatch and preparedStatement when usi…

2019-09-12 Thread GitBox
jinxing64 opened a new pull request #1452: [CALCITE-3338] Error with executeBatch and preparedStatement when usi… URL: https://github.com/apache/calcite/pull/1452 …ng RemoteMeta Parameter values are not passed into `CalciteConnectionImpl#enumerable` when using `RemoteMeta`. In curre

[GitHub] [calcite] XuQianJin-Stars commented on a change in pull request #1444: [CALCITE-3329] Implement osquery for OS adapter

2019-09-12 Thread GitBox
XuQianJin-Stars commented on a change in pull request #1444: [CALCITE-3329] Implement osquery for OS adapter URL: https://github.com/apache/calcite/pull/1444#discussion_r323202391 ## File path: plus/src/main/java/org/apache/calcite/adapter/os/CpuInfoTableFunction.java ##

[GitHub] [calcite] danny0405 commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG…

2019-09-12 Thread GitBox
danny0405 commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG… URL: https://github.com/apache/calcite/pull/1449#discussion_r323084636 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/MatchUtils.jav

[GitHub] [calcite] danny0405 commented on a change in pull request #1448: [CALCITE-3332] Query failed with AssertionError: cannot cast null as class java.math.BigDecima

2019-09-12 Thread GitBox
danny0405 commented on a change in pull request #1448: [CALCITE-3332] Query failed with AssertionError: cannot cast null as class java.math.BigDecima URL: https://github.com/apache/calcite/pull/1448#discussion_r323659868 ## File path: core/src/main/java/org/apache/calcite/sql/fun/S

[GitHub] [calcite] DonnyZone commented on a change in pull request #1448: [CALCITE-3332] Query failed with AssertionError: cannot cast null as class java.math.BigDecima

2019-09-12 Thread GitBox
DonnyZone commented on a change in pull request #1448: [CALCITE-3332] Query failed with AssertionError: cannot cast null as class java.math.BigDecima URL: https://github.com/apache/calcite/pull/1448#discussion_r323673338 ## File path: core/src/main/java/org/apache/calcite/sql/fun/S

[GitHub] [calcite] julianhyde commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG…

2019-09-12 Thread GitBox
julianhyde commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG… URL: https://github.com/apache/calcite/pull/1449#discussion_r323755129 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.j

[GitHub] [calcite] julianhyde commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG…

2019-09-12 Thread GitBox
julianhyde commented on a change in pull request #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG… URL: https://github.com/apache/calcite/pull/1449#discussion_r323756293 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/MatchUtils.ja

[GitHub] [calcite] julianhyde commented on issue #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG…

2019-09-12 Thread GitBox
julianhyde commented on issue #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG… URL: https://github.com/apache/calcite/pull/1449#issuecomment-530841447 Danny said that ‘@return’ comment should end in ‘.’. I disagree. The comment is a phrase, not a sentence. --

[GitHub] [calcite] zabetak commented on issue #552: [CALCITE-2018] Queries failed with AssertionError: rel has lower cost…

2019-09-12 Thread GitBox
zabetak commented on issue #552: [CALCITE-2018] Queries failed with AssertionError: rel has lower cost… URL: https://github.com/apache/calcite/pull/552#issuecomment-530842423 @xndai thanks for looking into this further. I kind of understand why it happens but I think it should not. -

[GitHub] [calcite] zabetak commented on issue #1439: [CALCITE-3322] There are two same testcase in RelMetadataTest

2019-09-12 Thread GitBox
zabetak commented on issue #1439: [CALCITE-3322] There are two same testcase in RelMetadataTest URL: https://github.com/apache/calcite/pull/1439#issuecomment-530903095 Closing as it was fixed in https://github.com/apache/calcite/commit/faeb6d0ea16c493157a1159c49cb970790c10cc1. ---

[GitHub] [calcite] zabetak closed pull request #1439: [CALCITE-3322] There are two same testcase in RelMetadataTest

2019-09-12 Thread GitBox
zabetak closed pull request #1439: [CALCITE-3322] There are two same testcase in RelMetadataTest URL: https://github.com/apache/calcite/pull/1439 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [calcite] hsyuan merged pull request #1442: [CALCITE-3319] AssertionError when reducing decimals

2019-09-12 Thread GitBox
hsyuan merged pull request #1442: [CALCITE-3319] AssertionError when reducing decimals URL: https://github.com/apache/calcite/pull/1442 This is an automated message from the Apache Git Service. To respond to the message, ple

[calcite] branch master updated (d3c7183 -> ce8ca67)

2019-09-12 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from d3c7183 [CALCITE-3331] Support implicit type cast for operators that use single operand family checker add ce8

[calcite] branch master updated (ce8ca67 -> 43b6abc)

2019-09-12 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from ce8ca67 [CALCITE-3319] AssertionError when reducing decimals (Wang Yanlin) add 43b6abc [CALCITE-3317] Add a p

[GitHub] [calcite] hsyuan merged pull request #1436: [CALCITE-3317] Add a public constructor for LogicalCalc with RelInput type parameter

2019-09-12 Thread GitBox
hsyuan merged pull request #1436: [CALCITE-3317] Add a public constructor for LogicalCalc with RelInput type parameter URL: https://github.com/apache/calcite/pull/1436 This is an automated message from the Apache Git Service

[GitHub] [calcite] hsyuan merged pull request #1433: [CALCITE-3316] Exception while deserializing LogicalCorrelate from json string

2019-09-12 Thread GitBox
hsyuan merged pull request #1433: [CALCITE-3316] Exception while deserializing LogicalCorrelate from json string URL: https://github.com/apache/calcite/pull/1433 This is an automated message from the Apache Git Service. To r

[calcite] branch master updated (43b6abc -> 507d383)

2019-09-12 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 43b6abc [CALCITE-3317] Add a public constructor for LogicalCalc with RelInput type parameter (Wang Yanlin) add

[GitHub] [calcite] JulianFeinauer commented on issue #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG…

2019-09-12 Thread GitBox
JulianFeinauer commented on issue #1449: [CALCITE-3302] Implement CLASSIFIER and LAST function for MATCH_RECOG… URL: https://github.com/apache/calcite/pull/1449#issuecomment-530931885 @julianhyde and @danny0405 thanks both of you for the comments. I fixed all of those and created another J

[calcite-avatica] branch master updated: Site: Add JavaScript client to client list

2019-09-12 Thread francischuang
This is an automated email from the ASF dual-hosted git repository. francischuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica.git The following commit(s) were added to refs/heads/master by this push: new 1ec1e81 Site: Add JavaScript cl

[calcite-site] branch master updated: Add JavaScript Avatica client to client list

2019-09-12 Thread francischuang
This is an automated email from the ASF dual-hosted git repository. francischuang 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 b6a511d Add JavaScript Avatica cli

[GitHub] [calcite] yanlin-Lynn commented on issue #1442: [CALCITE-3319] AssertionError when reducing decimals

2019-09-12 Thread GitBox
yanlin-Lynn commented on issue #1442: [CALCITE-3319] AssertionError when reducing decimals URL: https://github.com/apache/calcite/pull/1442#issuecomment-531073839 Thanks @hsyuan and @danny0405 ! This is an automated message