(datafusion-ballista) branch main updated: minor: enable json test (#1311)

[milenkovicm]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new be451795 minor: enable json test (#1311)
be451795 is described below

commit be45179570a7dc9954d4339a3c0b67578d3336ff
Author: Marko Milenković <milenkov...@users.noreply.github.com>
AuthorDate: Wed Sep 10 16:51:58 2025 +0100

    minor: enable json test (#1311)
---
 ballista/client/tests/context_checks.rs | 27 ++++++++++++---------------
 1 file changed, 12 insertions(+), 15 deletions(-)

diff --git a/ballista/client/tests/context_checks.rs 
b/ballista/client/tests/context_checks.rs
index 025cc4fb..a936f143 100644
--- a/ballista/client/tests/context_checks.rs
+++ b/ballista/client/tests/context_checks.rs
@@ -713,24 +713,21 @@ mod supported {
 
         assert_batches_eq!(expected, &result);
 
-        //
-        // TODO: enable when json supports lands in datafusion 47
-        //
-        // ctx.sql("select * from test")
-        //     .await?
-        //     .write_json(write_dir_path, Default::default(), 
Default::default())
-        //     .await?;
+        ctx.sql("select * from test")
+            .await?
+            .write_json(write_dir_path, Default::default(), Default::default())
+            .await?;
 
-        // ctx.register_json("j_written_table", write_dir_path, 
Default::default())
-        //     .await?;
+        ctx.register_json("j_written_table", write_dir_path, 
Default::default())
+            .await?;
 
-        // let result = ctx
-        //     .sql("select id, string_col from j_written_table where id > 4")
-        //     .await?
-        //     .collect()
-        //     .await?;
+        let result = ctx
+            .sql("select id, string_col from j_written_table where id > 4")
+            .await?
+            .collect()
+            .await?;
 
-        // assert_batches_eq!(expected, &result);
+        assert_batches_eq!(expected, &result);
 
         Ok(())
     }


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