asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259567401
##
File path:
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259567401
##
File path:
Hello!
Beam SQL uses Calcite's codegen for Calc relations. Ideally, we would
support nanosecond precision like java.sql.Timestamp and the new Java 8
date/time libraries. it looks like there is an assumption of millisecond
precision. Is this nanosecond support possible? I have not grokked the
Also, since you are particularly interested in applying filters from
one table on another, that is actually a nested loops join rather than
a filter. Linq4j implementation has examples of nested loops join in
the CorrelateJoinTest class.
Thanks,
Walaa.
No need to mention it, I also appreciate your contributions/discussions.
Block-based nested loops is not implemented but we already have tuple-based
nested loop [1] and it is a good starting point.
Regarding the particular query that you cited I think that even tuple-based
nested loop would work
kgyrtkirk opened a new pull request #1064: [CALCITE-2839] Simplify comparisions
against boolean literals
URL: https://github.com/apache/calcite/pull/1064
This patch enables to simplify x=true to x and x=false to not(x)
It also handles inequalities which are probably less frequent.
kgyrtkirk commented on a change in pull request #1053: [CALCITE-2852]
Simplification: traverse calculations and unknown UDFs
URL: https://github.com/apache/calcite/pull/1053#discussion_r259525126
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
Hi Gopal,
You may take a look at the FilterableTable API. It allows filtering tables
based on a list of RexNodes. Check
ScannableTableTest.testFilterableTableCoopertive() for an example test.
Thanks,
Walaa.
On Fri, Feb 22, 2019 at 12:37 PM Gopalakrishna Holla
wrote:
> Hello,
>
> The problem
julianhyde commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259516689
##
File path:
kgyrtkirk closed pull request #1047: [CALCITE-2841] Simplification: push
negation into Case expression
URL: https://github.com/apache/calcite/pull/1047
This is an automated message from the Apache Git Service.
To respond to
Yea I don't mind pushing out the RC towards the end of next week (beginning
of March). I'm not in a rush to build the RC. I just picked Monday to try
to hit the target of February that was arbitrarily picked in the sand. At
this point, the release will most likely happen in early March.
Kevin
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259513062
##
File path:
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259513062
##
File path:
Can you do the RC towards the end of next week?
I only just saw this message - 4 days after you sent it - because I’ve been
fighting down a backlog of 500 Apache messages. There are some changes I would
like to get into the release but I will need a few days.
Julian
> On Feb 18, 2019, at
julianhyde commented on a change in pull request #1053: [CALCITE-2852]
Simplification: traverse calculations and unknown UDFs
URL: https://github.com/apache/calcite/pull/1053#discussion_r259510833
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
julianhyde commented on a change in pull request #1053: [CALCITE-2852]
Simplification: traverse calculations and unknown UDFs
URL: https://github.com/apache/calcite/pull/1053#discussion_r259508694
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
julianhyde commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259507731
##
File path:
Hello,
The problem I'm working on is to order table scans so that one scan can
insert filters into the another. It seems like it should be a common
problem so I'm sure I'm missing some context. Taking a concrete example,
imagine two tables Users and Movies with the following schema
User {
id:
vineetgarg02 opened a new pull request #1063: CALCITE-1513 : Correlated NOT IN
query throws AssertionError(Vineet G…
URL: https://github.com/apache/calcite/pull/1063
…arg)
This is an automated message from the Apache Git
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259476211
##
File path:
Directory services are usually built-in. And that’s as they should be.
There’s nothing stopping you from creating schemas (or trees of schemas)
manually. And yes, you can put whatever kinds of objects you like as
constructor parameters for your schema. But if you want to use SchemaFactory,
or
Committers, please check the format of the commit comments. Rather than
CALCITE-2856-Emulating Cross Join For SparkSqlDialect
Should have been
[CALCITE-2856] Emulate Cross Join For SparkSqlDialect (Stuti Gupta)
> On Feb 21, 2019, at 5:06 PM, mm...@apache.org wrote:
>
> This is an
julianhyde commented on issue #1025: [CALCITE-2679] Fix a bug that `any_value`
can't be used with other ag…
URL: https://github.com/apache/calcite/pull/1025#issuecomment-466500691
Please change the commit comment to not start with 'fix'. Just provide the
description of the bug being
Actually there is a “contributors” group in JIRA and we can add people to it if
they ask.
But it’s also OK if someone (anyone - they don’t have to be a committer, just
have a JIRA account) writes into the JIRA case. “Hi, I’m working on this” and
maybe posts a link to their GitHub dev branch if
Rather than
> relBuilder.getRexBuilder().makeDateLiteral(new DateString(2019, 1, 1))
I think you can use
> relBuilder.literal(new DateString(2019, 1, 1))
(Or should be able to.) Same effect, a bit less code.
> On Feb 22, 2019, at 8:21 AM, Anton Haidai wrote:
>
> Stamatis, Hongze - thank
julianhyde commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259454491
##
File path:
zhztheplayer closed pull request #1013: [CALCITE-2791]Add the JSON_TYPE function
URL: https://github.com/apache/calcite/pull/1013
This is an automated message from the Apache Git Service.
To respond to the message, please
Yes it’s probably because I am the assignee.
I have bulk changed a lot of issues that were assigned to me, and made myself
no longer the default assignee for new issues. That will probably help.
Still, the JIRA-git integration is far too verbose in my opinion. If someone
adds a git comment
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259446058
##
File path:
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259446058
##
File path:
Julian - It might be because you are the default assignee for tickets? I
know I'm assigned tickets or watching them in other projects I get more
notifications.
Kevin Risden
On Fri, Feb 22, 2019 at 4:23 AM Francis Chuang
wrote:
> Would it make sense to have a separate list for Github
zabetak commented on a change in pull request #950: [CALCITE-2703] Reduce code
generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259442924
##
File path:
The Apache Jenkins build system has built Calcite-Master (build #1033)
Status: Failure
Check console output at https://builds.apache.org/job/Calcite-Master/1033/ to
view the results.
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259431543
##
File path:
kgyrtkirk commented on a change in pull request #1056: CALCITE-2858
URL: https://github.com/apache/calcite/pull/1056#discussion_r259429721
##
File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java
##
@@ -391,13 +385,17 @@ RexNode toRex(RelInput
zabetak merged pull request #898: [CALCITE-2464] Allow to set nullability for
columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898
This is an automated message from the Apache
zhztheplayer commented on issue #1004: [CALCITE-2786][CALCITE-2787] Add order
by clause support for JSON_ARRAYAGG; Json aggregate calls with different null
clause get in…
URL: https://github.com/apache/calcite/pull/1004#issuecomment-466460271
Thanks @michaelmior! 2 new PRs opened:
zabetak commented on a change in pull request #950: [CALCITE-2703] Reduce code
generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259417514
##
File path:
zhztheplayer closed pull request #1004: [CALCITE-2786][CALCITE-2787] Add order
by clause support for JSON_ARRAYAGG; Json aggregate calls with different null
clause get in…
URL: https://github.com/apache/calcite/pull/1004
Stamatis, Hongze - thank you for your help, it works now.
On Fri, Feb 22, 2019 at 6:05 PM Hongze Zhang wrote:
>
> >EnumerableCalc( expr#6=[TIMESTAMPADD($t3, $t4, $t5)] ...)
> >
> >But Calcite (both 1.18 and 1.19-SNAPSHOT) can't execute this plan:
> >
> >Caused by: java.lang.RuntimeException:
Mail format is converted to something bad :(
The example:
final RexNode shiftedDateField =
relBuilder.call(
SqlStdOperatorTable.DATETIME_PLUS,
rexBuilder.makeTimestampLiteral(new TimestampString(2019, 1, 1, 0, 0, 0), 0),
rexBuilder.makeCall(SqlStdOperatorTable.MULTIPLY,
asereda-gs commented on a change in pull request #950: [CALCITE-2703] Reduce
code generation and class loading overhead when executing queries in the
EnumerableConvention (Stamatis Zampetakis)
URL: https://github.com/apache/calcite/pull/950#discussion_r259409005
##
File path:
>EnumerableCalc( expr#6=[TIMESTAMPADD($t3, $t4, $t5)] ...)
>
>But Calcite (both 1.18 and 1.19-SNAPSHOT) can't execute this plan:
>
>Caused by: java.lang.RuntimeException: cannot translate call
>TIMESTAMPADD($t3, $t4, $t5)As @Stamatis has said, this is normal. Calcite
>doesn't provide an
Hi Anton,
The TIMESTAMPADD operator never ends up in the logical plan so essentially
there is no code for executing it. It is just used as a syntactic sugar in
the parser.
Have a look below to see how you can obtain the equivalent in relational
algebra:
yuqi1129 opened a new pull request #1062: [CALCITE-2861] Exception occurs when
do +-*/ between decimal and othe…
URL: https://github.com/apache/calcite/pull/1062
…r numeric types
This is an automated message from the Apache
Hongze, thank you for your answer. Your code successfully produces a
plan with the following node:
EnumerableCalc( expr#6=[TIMESTAMPADD($t3, $t4, $t5)] ...)
But Calcite (both 1.18 and 1.19-SNAPSHOT) can't execute this plan:
Caused by: java.lang.RuntimeException: cannot translate call
zhztheplayer opened a new pull request #1061: [CALCITE-2786] Add order by
clause support for JSON_ARRAYAGG
URL: https://github.com/apache/calcite/pull/1061
The JIRA issue:
https://issues.apache.org/jira/browse/CALCITE-2786
zhztheplayer opened a new pull request #1060: [CALCITE-2787] Json aggregate
calls with different null clause get in…
URL: https://github.com/apache/calcite/pull/1060
…correctly merged during converting from SQL to relational algebra
The JIRA issue:
https://cwiki.apache.org/confluence/display/INFRA/JDK+Installation+Matrix
That page has details about Jenkins INFRA structure stuff.
Kevin Risden
On Fri, Feb 22, 2019 at 9:14 AM Stamatis Zampetakis
wrote:
> BTW, which version of jdk1.8 we are using on Jenkins? It says latest but I
> cannot
zabetak commented on issue #898: [CALCITE-2464] Allow to set nullability for
columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898#issuecomment-466410857
@rubenada thanks for taking care of all the comments. The PR looks great:)
Can you please
BTW, which version of jdk1.8 we are using on Jenkins? It says latest but I
cannot find to which version latest really corresponds.
Is it this one jdk-8u201-linux-x64?
Στις Παρ, 22 Φεβ 2019 στις 1:59 μ.μ., ο/η Stamatis Zampetakis <
zabe...@gmail.com> έγραψε:
> Yes, I' ve noticed :( Let's keep
Correct the flag:
TimeUnitRange.MONTH -> TimeUnit.MONTH
Best,
Hongze
At 2019-02-22 22:07:11, "Hongze Zhang" wrote:
Hi Anton,
Following code works to me:
final RexNode shiftedDateField =
relBuilder.call(
SqlStdOperatorTable.TIMESTAMP_ADD,
Hi Anton,
Following code works to me:
final RexNode shiftedDateField =
relBuilder.call(
SqlStdOperatorTable.TIMESTAMP_ADD,
relBuilder.getRexBuilder().makeFlag(TimeUnitRange.MONTH),
relBuilder.literal(1),
relBuilder.getRexBuilder().makeDateLiteral(new
Issues can also be assigned to contributors and I've added you as a
contributor and assigned the issue to you. Thanks!
--
Michael Mior
mm...@apache.org
Le jeu. 21 févr. 2019 à 12:56, Ilia Gorelikhin
a écrit :
>
> Hello! I would like to help with task
>
Yes, I' ve noticed :( Let's keep looking!
Στις Πέμ, 21 Φεβ 2019 στις 12:48 μ.μ., ο/η Vladimir Sitnikov <
sitnikov.vladi...@gmail.com> έγραψε:
> I've performed multiple builds, and it looks like Java8 is still suffering
> from misunderoptimization :(
>
michaelmior commented on issue #1004: [CALCITE-2786][CALCITE-2787] Add order by
clause support for JSON_ARRAYAGG; Json aggregate calls with different null
clause get in…
URL: https://github.com/apache/calcite/pull/1004#issuecomment-466387868
These seem to be two separate issues. Could you
zabetak commented on a change in pull request #898: [CALCITE-2464] Allow to set
nullability for columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898#discussion_r259330299
##
File path:
michaelmior commented on issue #1016: [CALCITE-2803] ProjectTransposeJoinRule
messes INDF expressions
URL: https://github.com/apache/calcite/pull/1016#issuecomment-466386868
@laurentgo Could you fix the import order issue that's causing checkstyle to
fail? (See the Travis tests.)
zabetak commented on a change in pull request #898: [CALCITE-2464] Allow to set
nullability for columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898#discussion_r259328399
##
File path:
zabetak commented on a change in pull request #898: [CALCITE-2464] Allow to set
nullability for columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898#discussion_r259328901
##
File path:
michaelmior commented on a change in pull request #1025: [CALCITE-2679] Fix a
bug that `any_value` can't be used with other ag…
URL: https://github.com/apache/calcite/pull/1025#discussion_r259329347
##
File path:
zabetak commented on a change in pull request #898: [CALCITE-2464] Allow to set
nullability for columns of structured types (Ruben Quesada Lopez)
URL: https://github.com/apache/calcite/pull/898#discussion_r259329627
##
File path:
michaelmior commented on issue #1025: [CALCITE-2679] Fix a bug that `any_value`
can't be used with other ag…
URL: https://github.com/apache/calcite/pull/1025#issuecomment-466386448
@asereda-gs Do you mind having a look at this?
yuqi1129 commented on a change in pull request #1057: [CALCITE-2854] code gen
error for UNARY_MINUS operator call with deci…
URL: https://github.com/apache/calcite/pull/1057#discussion_r259329092
##
File path: core/src/test/java/org/apache/calcite/test/ReflectiveSchemaTest.java
Hello. In SQL, I can execute a query like "SELECT TIMESTAMPADD(month,
1, "date" ) FROM ..." and it works. But my attempts to do the same
thing using RelBuilder are not successful, for example, this code does
not work:
RexNode shiftedDateField = relBuilder.call(
stutigupta028 commented on a change in pull request #1052: CALCITE-2833- Select
without from support & CALCITE-2834-Support for values clause for dialects
which neither have values clause nor dual table
URL: https://github.com/apache/calcite/pull/1052#discussion_r259298635
##
File
zabetak opened a new pull request #1059: [CALCITE-2859] Centralize Calcite
system properties
URL: https://github.com/apache/calcite/pull/1059
This is an automated message from the Apache Git Service.
To respond to the
Hi Ilia,
I have the impression that issues can be assigned only to committers of the
project.
Nevertheless, you can definetely work on this and submit a PR. Tickets are
not necessary resolved by the people assigned to them.
Best,
Stamatis
Στις Πέμ, 21 Φεβ 2019 στις 6:56 μ.μ., ο/η Ilia
vlsi commented on a change in pull request #1057: [CALCITE-2854] code gen error
for UNARY_MINUS operator call with deci…
URL: https://github.com/apache/calcite/pull/1057#discussion_r259274565
##
File path: core/src/test/java/org/apache/calcite/test/ReflectiveSchemaTest.java
yuqi1129 commented on a change in pull request #1057: [CALCITE-2854] code gen
error for UNARY_MINUS operator call with deci…
URL: https://github.com/apache/calcite/pull/1057#discussion_r259272042
##
File path: core/src/test/java/org/apache/calcite/test/JdbcTest.java
##
@@
Would it make sense to have a separate list for Github notifications?
This would be similar to the commits@ list but would be dedicated to
archiving those Github notifications.
On 22/02/2019 6:44 pm, Vladimir Sitnikov wrote:
Julian>I’m getting a dozen or so such emails each day. It seems that
yuqi created CALCITE-2861:
-
Summary: Exception occurs when do +-*/ between decimal and other
numeric types
Key: CALCITE-2861
URL: https://issues.apache.org/jira/browse/CALCITE-2861
Project: Calcite
72 matches
Mail list logo