(arrow-datafusion) branch main updated: [Minor] Update TCPDS tests, remove some #[ignore]d tests (#9829)

[alamb]

This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 656887c115 [Minor] Update TCPDS tests, remove some #[ignore]d tests 
(#9829)
656887c115 is described below

commit 656887c1158e6855da54407986c710b8006968fd
Author: Daniël Heres <danielhe...@gmail.com>
AuthorDate: Thu Mar 28 18:04:53 2024 +0100

    [Minor] Update TCPDS tests, remove some #[ignore]d tests (#9829)
    
    * Update TCPDS tests
    
    * Fmt
---
 datafusion/core/tests/tpcds_planning.rs | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/datafusion/core/tests/tpcds_planning.rs 
b/datafusion/core/tests/tpcds_planning.rs
index 4db97c75cb..e8d2c3764e 100644
--- a/datafusion/core/tests/tpcds_planning.rs
+++ b/datafusion/core/tests/tpcds_planning.rs
@@ -73,9 +73,6 @@ async fn tpcds_logical_q9() -> Result<()> {
     create_logical_plan(9).await
 }
 
-#[ignore]
-// Schema error: No field named 'c'.'c_customer_sk'.
-// issue: https://github.com/apache/arrow-datafusion/issues/4794
 #[tokio::test]
 async fn tpcds_logical_q10() -> Result<()> {
     create_logical_plan(10).await
@@ -201,9 +198,6 @@ async fn tpcds_logical_q34() -> Result<()> {
     create_logical_plan(34).await
 }
 
-#[ignore]
-// Schema error: No field named 'c'.'c_customer_sk'.
-// issue: https://github.com/apache/arrow-datafusion/issues/4794
 #[tokio::test]
 async fn tpcds_logical_q35() -> Result<()> {
     create_logical_plan(35).await
@@ -577,7 +571,7 @@ async fn tpcds_physical_q9() -> Result<()> {
     create_physical_plan(9).await
 }
 
-#[ignore] // FieldNotFound
+#[ignore] // Physical plan does not support logical expression 
Exists(<subquery>)
 #[tokio::test]
 async fn tpcds_physical_q10() -> Result<()> {
     create_physical_plan(10).await
@@ -703,7 +697,7 @@ async fn tpcds_physical_q34() -> Result<()> {
     create_physical_plan(34).await
 }
 
-#[ignore] // FieldNotFound
+#[ignore] // Physical plan does not support logical expression 
Exists(<subquery>)
 #[tokio::test]
 async fn tpcds_physical_q35() -> Result<()> {
     create_physical_plan(35).await
@@ -734,7 +728,8 @@ async fn tpcds_physical_q40() -> Result<()> {
     create_physical_plan(40).await
 }
 
-#[ignore] // Physical plan does not support logical expression (<subquery>)
+#[ignore]
+// Context("check_analyzed_plan", Plan("Correlated column is not allowed in 
predicate: (..)
 #[tokio::test]
 async fn tpcds_physical_q41() -> Result<()> {
     create_physical_plan(41).await