[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Julian Hyde (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15860087#comment-15860087
 ] 

Julian Hyde commented on CALCITE-1625:
--

You mentioned {{getImportedKeys}}, which is about foreign keys. Anyway, 
CALCITE-1545 covers both primary and foreign keys. Maybe the first commit only 
fixes part of it.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Jess Balint (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15860078#comment-15860078
 ] 

Jess Balint commented on CALCITE-1625:
--

I am talking about getting _primary_ keys and table cardinalities during 
{{JdbcSchema.computeTables()}} (or lazily) and exposing it in the {{Statistic}} 
in the {{JdbcTable}}.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15860003#comment-15860003
 ] 

Josh Elser commented on CALCITE-1625:
-

bq. I'm primarily interested in adding statistics under JdbcTable. Afaik, this 
is not related to Avatica

Ok, sorry if I'm derailing this :). I assumed it was going through the Calcite 
JDBC driver which would inherit the problem. If it's using any JDBC driver, 
you're right, it's unrelated.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Jess Balint (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1586#comment-1586
 ] 

Jess Balint commented on CALCITE-1625:
--

It is certainly related. I can push up the patch I have which copies basic 
stats from {{DatabaseMetaData.getIndexInfo()}} to the already present 
{{Statistic}} structure. The scope of CALCITE-1545 is not clear. We should 
clarify the scope of CALCITE-1545 to build a representation of foreign keys (or 
maybe generalized dependencies) and retrieve that via 
{{DatabaseMetaData.getImportedKeys()}}. Possibly two separate tasks.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Julian Hyde (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15859991#comment-15859991
 ] 

Julian Hyde commented on CALCITE-1625:
--

Is this a duplicate of CALCITE-1545?

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Jess Balint (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15859970#comment-15859970
 ] 

Jess Balint commented on CALCITE-1625:
--

I'm primarily interested in adding statistics under {{JdbcTable}}. Afaik, this 
is not related to Avatica. When using JDBC tables in planning, it's very 
helpful to know the keys and cardinality.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (CALCITE-1625) add basic JDBC statistics

2017-02-09 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/CALCITE-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15859932#comment-15859932
 ] 

Josh Elser commented on CALCITE-1625:
-

Beware that we're probably going to also run into CALCITE-1308 as a blocker 
before we could do this one.

> add basic JDBC statistics
> -
>
> Key: CALCITE-1625
> URL: https://issues.apache.org/jira/browse/CALCITE-1625
> Project: Calcite
>  Issue Type: Improvement
>  Components: core
>Reporter: Jess Balint
>Assignee: Julian Hyde
>Priority: Minor
>
> basic JDBC statistics can be retrieved via 
> {{DatabaseMetaData.getIndexInfo()}}. This can be used in the underlying 
> {{JdbcTable}} to provide stats/key info during planning.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)