[jira] [Commented] (CALCITE-1598) Pig adapter

[ https://issues.apache.org/jira/browse/CALCITE-1598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887449#comment-15887449 ] Eli Levine commented on CALCITE-1598: - [~julianhyde], the documentation page looks g

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887372#comment-15887372 ] Ashutosh Chauhan commented on CALCITE-1661: --- I agree if column is declared as

[jira] [Commented] (CALCITE-1655) Add In filter to druid adapter.

[ https://issues.apache.org/jira/browse/CALCITE-1655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887314#comment-15887314 ] Ashutosh Chauhan commented on CALCITE-1655: --- I dont think its a good idea to f

[jira] [Resolved] (CALCITE-1660) SqlNameMatcher does not match fields in a structure

[ https://issues.apache.org/jira/browse/CALCITE-1660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-1660. -- Resolution: Not A Problem SqlNameMatcher has a simple purpose: to match names, applying a part

[jira] [Comment Edited] (CALCITE-1664) CAST('' as TIMESTAMP) adds part of sub-second fraction to the value

[ https://issues.apache.org/jira/browse/CALCITE-1664?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887172#comment-15887172 ] Julian Hyde edited comment on CALCITE-1664 at 2/28/17 3:28 AM: ---

[jira] [Resolved] (CALCITE-1664) CAST('' as TIMESTAMP) adds part of sub-second fraction to the value

[ https://issues.apache.org/jira/browse/CALCITE-1664?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-1664. -- Resolution: Fixed Fix Version/s: avatica-1.10.0 Fixed in http://git-wip-us.apache.org/r

[jira] [Commented] (CALCITE-1510) INSERT/UPSERT should allow fewer values than columns

[ https://issues.apache.org/jira/browse/CALCITE-1510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887168#comment-15887168 ] Julian Hyde commented on CALCITE-1510: -- Addendum committed as http://git-wip-us.ap

[jira] [Resolved] (CALCITE-1615) Support HOP and SESSION in the GROUP BY clause

[ https://issues.apache.org/jira/browse/CALCITE-1615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-1615. -- Resolution: Fixed Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/49888a6c. Tha

[jira] [Commented] (CALCITE-1598) Pig adapter

[ https://issues.apache.org/jira/browse/CALCITE-1598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887163#comment-15887163 ] Julian Hyde commented on CALCITE-1598: -- I have reviewed, and am almost ready to che

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887034#comment-15887034 ] Julian Hyde commented on CALCITE-1661: -- I don't think we should support DECIMAL in

[jira] [Commented] (CALCITE-1655) Add In filter to druid adapter.

[ https://issues.apache.org/jira/browse/CALCITE-1655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886993#comment-15886993 ] Julian Hyde commented on CALCITE-1655: -- IN and BETWEEN are not valid operators in t

[jira] [Commented] (CALCITE-1656) Sub-Optimal Druid Query planning - Does not Prune columns for DruidQuery

[ https://issues.apache.org/jira/browse/CALCITE-1656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886951#comment-15886951 ] Julian Hyde commented on CALCITE-1656: -- [~nishantbangarwa], A couple of review comm

[jira] [Comment Edited] (CALCITE-1615) Support HOP and SESSION in the GROUP BY clause

[ https://issues.apache.org/jira/browse/CALCITE-1615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886802#comment-15886802 ] Haohui Mai edited comment on CALCITE-1615 at 2/28/17 12:16 AM: ---

[jira] [Commented] (CALCITE-1615) Support HOP and SESSION in the GROUP BY clause

[ https://issues.apache.org/jira/browse/CALCITE-1615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886802#comment-15886802 ] Haohui Mai commented on CALCITE-1615: - That sounds good to me. Please go ahead. > S

[jira] [Commented] (CALCITE-1615) Support HOP and SESSION in the GROUP BY clause

[ https://issues.apache.org/jira/browse/CALCITE-1615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886801#comment-15886801 ] Julian Hyde commented on CALCITE-1615: -- [~wheat9], I saw you added non-reserved key

[jira] [Commented] (CALCITE-1515) Support TableFunctionScan in RelBuilder

[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886589#comment-15886589 ] Julian Hyde commented on CALCITE-1515: -- Sorry, I misread {{RexNode}} as {{RelNode}}

[jira] [Updated] (CALCITE-1510) INSERT/UPSERT should allow fewer values than columns

[ https://issues.apache.org/jira/browse/CALCITE-1510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-1510: - Summary: INSERT/UPSERT should allow fewer values than columns (was: INSERT/UPSERT should allow

[jira] [Commented] (CALCITE-1353) first_frame_max_size in an ExecuteRequest should be an int32 in protobuf definitions.

[ https://issues.apache.org/jira/browse/CALCITE-1353?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886538#comment-15886538 ] Francis Chuang commented on CALCITE-1353: - I am going to close that PR as it bre

[jira] [Commented] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886530#comment-15886530 ] Julian Hyde commented on CALCITE-1659: -- Fixed in http://git-wip-us.apache.org/repo

[jira] [Resolved] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde resolved CALCITE-1659. -- Resolution: Fixed > Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-secon

[jira] [Commented] (CALCITE-1613) Implement EXTRACT for time unit DOW, DOY; and fix CENTURY

[ https://issues.apache.org/jira/browse/CALCITE-1613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886525#comment-15886525 ] Julian Hyde commented on CALCITE-1613: -- Also update the test case for CALCITE-1664,

[jira] [Commented] (CALCITE-1612) Release Avatica 1.10

[ https://issues.apache.org/jira/browse/CALCITE-1612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886520#comment-15886520 ] Julian Hyde commented on CALCITE-1612: -- After upgrading Calcite to Avatica-1.10, up

[jira] [Comment Edited] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886437#comment-15886437 ] Remus Rusanu edited comment on CALCITE-1659 at 2/27/17 8:06 PM: --

[jira] [Comment Edited] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886437#comment-15886437 ] Remus Rusanu edited comment on CALCITE-1659 at 2/27/17 8:06 PM: --

[jira] [Commented] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886437#comment-15886437 ] Remus Rusanu commented on CALCITE-1659: --- > Seems you figured out how to run Quidem

[jira] [Comment Edited] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886420#comment-15886420 ] Jesus Camacho Rodriguez edited comment on CALCITE-1661 at 2/27/17 8:04 PM: ---

[jira] [Comment Edited] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886420#comment-15886420 ] Jesus Camacho Rodriguez edited comment on CALCITE-1661 at 2/27/17 7:58 PM: ---

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886420#comment-15886420 ] Jesus Camacho Rodriguez commented on CALCITE-1661: -- You are right, that

[jira] [Commented] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886392#comment-15886392 ] Remus Rusanu commented on CALCITE-1659: --- When you commit, please also push new sna

[jira] [Updated] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-1659: - Fix Version/s: 1.12.0 > Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-sec

[jira] [Updated] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-1659: - Description: Timestamps are losing sub-second parts after Calcite literal constant reduction. {

[jira] [Updated] (CALCITE-1659) Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Julian Hyde updated CALCITE-1659: - Summary: Simplifying CAST(characterLiteral as TIMESTAMP) should round the sub-second fraction (

[jira] [Commented] (CALCITE-1659) Timestamps are loosing sub-second parts after Calcite literal constant reduction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886374#comment-15886374 ] Julian Hyde commented on CALCITE-1659: -- Reviewing the code now. It seems to make se

[jira] [Commented] (CALCITE-1659) Timestamps are loosing sub-second parts after Calcite literal constant reduction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886357#comment-15886357 ] Julian Hyde commented on CALCITE-1659: -- {quote} Anywhere I can get quickly up to sp

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886336#comment-15886336 ] Julian Hyde commented on CALCITE-1661: -- So, if Hive has a column c of type {{DECIMA

[jira] [Commented] (CALCITE-1663) TIMESTAMPDIFF declares operand type DATETIME instead of TIMESTMAP

[ https://issues.apache.org/jira/browse/CALCITE-1663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886326#comment-15886326 ] Julian Hyde commented on CALCITE-1663: -- My goal was to do as little as possible, na

[jira] [Commented] (CALCITE-1663) TIMESTAMPDIFF declares operand type DATETIME instead of TIMESTMAP

[ https://issues.apache.org/jira/browse/CALCITE-1663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886305#comment-15886305 ] Remus Rusanu commented on CALCITE-1663: --- I don't have a strong opinion on this. I

[jira] [Commented] (CALCITE-1353) first_frame_max_size in an ExecuteRequest should be an int32 in protobuf definitions.

[ https://issues.apache.org/jira/browse/CALCITE-1353?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886284#comment-15886284 ] Julian Hyde commented on CALCITE-1353: -- [~elserj] Can I have your input on this? Th

[jira] [Commented] (CALCITE-1662) TimestampDiffConvertlet uses second resolution even for sub-second units

[ https://issues.apache.org/jira/browse/CALCITE-1662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886277#comment-15886277 ] Remus Rusanu commented on CALCITE-1662: --- Well, MySQL {{select TIMESTAMP '2016-02-2

[jira] [Comment Edited] (CALCITE-1662) TimestampDiffConvertlet uses second resolution even for sub-second units

[ https://issues.apache.org/jira/browse/CALCITE-1662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886254#comment-15886254 ] Remus Rusanu edited comment on CALCITE-1662 at 2/27/17 6:24 PM: --

[jira] [Commented] (CALCITE-1662) TimestampDiffConvertlet uses second resolution even for sub-second units

[ https://issues.apache.org/jira/browse/CALCITE-1662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886254#comment-15886254 ] Remus Rusanu commented on CALCITE-1662: --- My initial tests for CALCITE-1659 contain

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886205#comment-15886205 ] Jesus Camacho Rodriguez commented on CALCITE-1661: -- [~julianhyde], I ha

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886139#comment-15886139 ] Julian Hyde commented on CALCITE-1661: -- I see that [FRACTIONAL_TYPES|https://githu

[jira] [Commented] (CALCITE-1662) TimestampDiffConvertlet uses second resolution even for sub-second units

[ https://issues.apache.org/jira/browse/CALCITE-1662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886133#comment-15886133 ] Julian Hyde commented on CALCITE-1662: -- I can't quite see where in the code the nai

[jira] [Commented] (CALCITE-1664) CAST('' as TIMESTAMP) adds part of sub-second fraction to the value

[ https://issues.apache.org/jira/browse/CALCITE-1664?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886119#comment-15886119 ] Remus Rusanu commented on CALCITE-1664: --- {{CAST('2016-02-26 19:06:00.123456789' as

[jira] [Commented] (CALCITE-1663) TIMESTAMPDIFF declares operand type DATETIME instead of TIMESTMAP

[ https://issues.apache.org/jira/browse/CALCITE-1663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886117#comment-15886117 ] Julian Hyde commented on CALCITE-1663: -- What do you consider to be the definitive s

[jira] [Commented] (CALCITE-1664) CAST('' as TIMESTAMP) adds part of sub-second fraction to the value

[ https://issues.apache.org/jira/browse/CALCITE-1664?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886111#comment-15886111 ] Julian Hyde commented on CALCITE-1664: -- I don't see a problem in the output above.

[jira] [Commented] (CALCITE-1659) Timestamps are loosing sub-second parts after Calcite literal constant reduction

[ https://issues.apache.org/jira/browse/CALCITE-1659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885968#comment-15885968 ] Remus Rusanu commented on CALCITE-1659: --- I added test case. I also added the sam

[jira] [Created] (CALCITE-1664) CAST('' as TIMESTAMP) adds part of sub-second fraction to the value

Remus Rusanu created CALCITE-1664: - Summary: CAST('' as TIMESTAMP) adds part of sub-second fraction to the value Key: CALCITE-1664 URL: https://issues.apache.org/jira/browse/CALCITE-1664 Project: Calc

[jira] [Created] (CALCITE-1663) TIMESTAMPDIFF declares operand type DATETIME instead of TIMESTMAP

Remus Rusanu created CALCITE-1663: - Summary: TIMESTAMPDIFF declares operand type DATETIME instead of TIMESTMAP Key: CALCITE-1663 URL: https://issues.apache.org/jira/browse/CALCITE-1663 Project: Calcit

[jira] [Created] (CALCITE-1662) TimestampDiffConvertlet uses second resolution even for sub-second units

Remus Rusanu created CALCITE-1662: - Summary: TimestampDiffConvertlet uses second resolution even for sub-second units Key: CALCITE-1662 URL: https://issues.apache.org/jira/browse/CALCITE-1662 Project:

[jira] [Commented] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

[ https://issues.apache.org/jira/browse/CALCITE-1661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885564#comment-15885564 ] Jesus Camacho Rodriguez commented on CALCITE-1661: -- [~julianhyde], coul

[jira] [Commented] (CALCITE-1655) Add In filter to druid adapter.

[ https://issues.apache.org/jira/browse/CALCITE-1655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885552#comment-15885552 ] Jesus Camacho Rodriguez commented on CALCITE-1655: -- IN and BETWEEN are

[jira] [Created] (CALCITE-1661) Recognize aggregation function types as FRACTIONAL instead of DOUBLE

Jesus Camacho Rodriguez created CALCITE-1661: Summary: Recognize aggregation function types as FRACTIONAL instead of DOUBLE Key: CALCITE-1661 URL: https://issues.apache.org/jira/browse/CALCITE-1661

[jira] [Comment Edited] (CALCITE-1515) Support TableFunctionScan in RelBuilder

[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885351#comment-15885351 ] Anton Mushin edited comment on CALCITE-1515 at 2/27/17 8:35 AM: --

[jira] [Comment Edited] (CALCITE-1515) Support TableFunctionScan in RelBuilder

[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885351#comment-15885351 ] Anton Mushin edited comment on CALCITE-1515 at 2/27/17 8:35 AM: --

[jira] [Commented] (CALCITE-1515) Support TableFunctionScan in RelBuilder

[ https://issues.apache.org/jira/browse/CALCITE-1515?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885351#comment-15885351 ] Anton Mushin commented on CALCITE-1515: --- bq.Regarding 2: use ImmutableList.copyOf(