[jira] [Commented] (SPARK-37199) Add a deterministic field to QueryPlan

2021-11-02 Thread Apache Spark (Jira)


[ 
https://issues.apache.org/jira/browse/SPARK-37199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17437527#comment-17437527
 ] 

Apache Spark commented on SPARK-37199:
--

User 'somani' has created a pull request for this issue:
https://github.com/apache/spark/pull/34470

> Add a deterministic field to QueryPlan
> --
>
> Key: SPARK-37199
> URL: https://issues.apache.org/jira/browse/SPARK-37199
> Project: Spark
>  Issue Type: Improvement
>  Components: SQL
>Affects Versions: 3.2.0
>Reporter: Abhishek Somani
>Priority: Major
>
> We have a _deterministic_ field in 
> [Expressions|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala#L115]
>  to check if an expression is deterministic, but we do not have a similar 
> field in 
> [QueryPlan.|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala#L44]
> We have a need for such a check in the QueryPlan sometimes, like in 
> [InlineCTE|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/InlineCTE.scala#L56]
> This proposal is to add a _deterministic_ field to QueryPlan.
> More details [in this 
> document|https://docs.google.com/document/d/1eIiaSJf-Co2HhjsaQxFNGwUxobnHID4ZGmJMcVytREc/edit#].



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

-
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org



[jira] [Commented] (SPARK-37199) Add a deterministic field to QueryPlan

2021-11-02 Thread Abhishek Somani (Jira)


[ 
https://issues.apache.org/jira/browse/SPARK-37199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17437514#comment-17437514
 ] 

Abhishek Somani commented on SPARK-37199:
-

Will add a PR soon.

> Add a deterministic field to QueryPlan
> --
>
> Key: SPARK-37199
> URL: https://issues.apache.org/jira/browse/SPARK-37199
> Project: Spark
>  Issue Type: Improvement
>  Components: SQL
>Affects Versions: 3.2.0
>Reporter: Abhishek Somani
>Priority: Major
>
> We have a _deterministic_ field in 
> [Expressions|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala#L115]
>  to check if an expression is deterministic, but we do not have a similar 
> field in 
> [QueryPlan.|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala#L44]
> We have a need for such a check in the QueryPlan sometimes, like in 
> [InlineCTE|https://github.com/apache/spark/blob/b78167a2ee6b11b1f2839274e23676411f919115/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/InlineCTE.scala#L56]
> This proposal is to add a _deterministic_ field to QueryPlan.
> More details [in this 
> document|https://docs.google.com/document/d/1eIiaSJf-Co2HhjsaQxFNGwUxobnHID4ZGmJMcVytREc/edit#].



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

-
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org