[jira] [Updated] (CALCITE-3673) ListTransientTable should not leave tables in the schema

2022-01-13 Thread Ruben Q L (Jira)


 [ 
https://issues.apache.org/jira/browse/CALCITE-3673?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ruben Q L updated CALCITE-3673:
---
Fix Version/s: 1.30.0

> ListTransientTable should not leave tables in the schema
> 
>
> Key: CALCITE-3673
> URL: https://issues.apache.org/jira/browse/CALCITE-3673
> Project: Calcite
>  Issue Type: New Feature
>  Components: core
>Affects Versions: 1.21.0
>Reporter: Vladimir Sitnikov
>Assignee: Ruben Q L
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.30.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> 1) org.apache.calcite.schema.impl.ListTransientTable#scan adds itself to a 
> schema: root.getRootSchema().add(name, this); however, it misses cleanup, so 
> it looks like a memory leak
> 2) ListTransientTable is not implementable with EnumerableTableScan as 
> org.apache.calcite.tools.RelBuilder#transientScan(java.lang.String, 
> org.apache.calcite.rel.type.RelDataType) is using 
> org.apache.calcite.prepare.RelOptTableImpl#create(org.apache.calcite.plan.RelOptSchema,
>  org.apache.calcite.rel.type.RelDataType, org.apache.calcite.schema.Table, 
> com.google.common.collect.ImmutableList) which creates 
> RelOptTableImpl with NULL expressionFunction which means 
> org.apache.calcite.prepare.RelOptTableImpl#getExpression returns null



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Updated] (CALCITE-3673) ListTransientTable should not leave tables in the schema

2020-01-03 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/CALCITE-3673?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

ASF GitHub Bot updated CALCITE-3673:

Labels: pull-request-available  (was: )

> ListTransientTable should not leave tables in the schema
> 
>
> Key: CALCITE-3673
> URL: https://issues.apache.org/jira/browse/CALCITE-3673
> Project: Calcite
>  Issue Type: New Feature
>  Components: core
>Affects Versions: 1.21.0
>Reporter: Vladimir Sitnikov
>Priority: Major
>  Labels: pull-request-available
>
> 1) org.apache.calcite.schema.impl.ListTransientTable#scan adds itself to a 
> schema: root.getRootSchema().add(name, this); however, it misses cleanup, so 
> it looks like a memory leak
> 2) ListTransientTable is not implementable with EnumerableTableScan as 
> org.apache.calcite.tools.RelBuilder#transientScan(java.lang.String, 
> org.apache.calcite.rel.type.RelDataType) is using 
> org.apache.calcite.prepare.RelOptTableImpl#create(org.apache.calcite.plan.RelOptSchema,
>  org.apache.calcite.rel.type.RelDataType, org.apache.calcite.schema.Table, 
> com.google.common.collect.ImmutableList) which creates 
> RelOptTableImpl with NULL expressionFunction which means 
> org.apache.calcite.prepare.RelOptTableImpl#getExpression returns null



--
This message was sent by Atlassian Jira
(v8.3.4#803005)