[jira] [Commented] (CALCITE-6251) innerEnumerator in EnumerableDefaults::correlateBatchJoin is not closed

2024-02-07 Thread Ulrich Kramer (Jira)


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

Ulrich Kramer commented on CALCITE-6251:


It's already there.

> innerEnumerator in EnumerableDefaults::correlateBatchJoin is not closed
> ---
>
> Key: CALCITE-6251
> URL: https://issues.apache.org/jira/browse/CALCITE-6251
> Project: Calcite
>  Issue Type: Bug
>Affects Versions: 1.36.0
>Reporter: Ulrich Kramer
>Priority: Major
>  Labels: pull-request-available
>
> The 
> [innerEnumerator|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1681]
>  is only closed at the end. But if there are multiple loops, [innerEnumerator 
> is just assigned to a different value without closing 
> it|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1720].
> It should look like 
> [here|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1547-L1550].



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (CALCITE-6251) innerEnumerator in EnumerableDefaults::correlateBatchJoin is not closed

2024-02-07 Thread Ruben Q L (Jira)


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

Ruben Q L commented on CALCITE-6251:


Well spotted [~kramerul]! Would you open a PR to fix this?

> innerEnumerator in EnumerableDefaults::correlateBatchJoin is not closed
> ---
>
> Key: CALCITE-6251
> URL: https://issues.apache.org/jira/browse/CALCITE-6251
> Project: Calcite
>  Issue Type: Bug
>Affects Versions: 1.36.0
>Reporter: Ulrich Kramer
>Priority: Major
>
> The 
> [innerEnumerator|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1681]
>  is only closed at the end. But if there are multiple loops, [innerEnumerator 
> is just assigned to a different value without closing 
> it|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1720].
> It should look like 
> [here|https://github.com/apache/calcite/blob/f7069cc5245c22f816c565669f52b4f30b046f4d/linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java#L1547-L1550].



--
This message was sent by Atlassian Jira
(v8.20.10#820010)