You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datafusion.apache.org by gi...@apache.org on 2026/09/24 19:23:19 UTC

(datafusion) 01/01: docs: Remove extra commas in count results (#25552)

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

github-merge-queue[bot] pushed a commit to branch gh-readonly-queue/main/pr-25552-db20a1ebe50fda78f30d7f5d8a418cd7c4c83eff
in repository https://gitbox.apache.org/repos/asf/datafusion.git

commit af86b375d1a52c7118b42010318ceb7d58ac2430
Author: Nuno Faria <nu...@gmail.com>
AuthorDate: Thu Sep 24 19:22:59 2026 +0000

    docs: Remove extra commas in count results (#25552)
    
    ## Which issue does this PR close?
    
    - N/A.
    
    ## Rationale for this change
    
    Noticed that the docs for the `TableProvider` had extra commas in output
    of a dataframe. Turns out this shows up in two other places.
    
    ## What changes are included in this PR?
    
    - Remove unnecessary commas.
    
    ## What is the testing strategy for this PR?
    
    N/A.
    
    ## Are there any user-facing changes?
    
    No.
---
 datafusion/core/src/datasource/memory_test.rs | 10 +++++-----
 datafusion/datasource/src/sink.rs             | 10 +++++-----
 datafusion/session/src/table.rs               | 10 +++++-----
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/datafusion/core/src/datasource/memory_test.rs b/datafusion/core/src/datasource/memory_test.rs
index ff464ec3d3..c7b8e06b34 100644
--- a/datafusion/core/src/datasource/memory_test.rs
+++ b/datafusion/core/src/datasource/memory_test.rs
@@ -377,11 +377,11 @@ mod tests {
     /// Returns the value of results. For example, returns 6 given the following
     ///
     /// ```text
-    /// +-------+,
-    /// | count |,
-    /// +-------+,
-    /// | 6     |,
-    /// +-------+,
+    /// +-------+
+    /// | count |
+    /// +-------+
+    /// | 6     |
+    /// +-------+
     /// ```
     fn extract_count(res: Vec<RecordBatch>) -> u64 {
         assert_eq!(res.len(), 1, "expected one batch, got {}", res.len());
diff --git a/datafusion/datasource/src/sink.rs b/datafusion/datasource/src/sink.rs
index 1557e41531..6851c9d7f4 100644
--- a/datafusion/datasource/src/sink.rs
+++ b/datafusion/datasource/src/sink.rs
@@ -385,11 +385,11 @@ impl ExecutionPlan for DataSinkExec {
 /// Create a output record batch with a count
 ///
 /// ```text
-/// +-------+,
-/// | count |,
-/// +-------+,
-/// | 6     |,
-/// +-------+,
+/// +-------+
+/// | count |
+/// +-------+
+/// | 6     |
+/// +-------+
 /// ```
 fn make_count_batch(count: u64) -> RecordBatch {
     let array = Arc::new(UInt64Array::from(vec![count])) as ArrayRef;
diff --git a/datafusion/session/src/table.rs b/datafusion/session/src/table.rs
index 112d281741..999ec16b89 100644
--- a/datafusion/session/src/table.rs
+++ b/datafusion/session/src/table.rs
@@ -335,11 +335,11 @@ pub trait TableProvider: Any + Debug + Sync + Send {
     /// column called "count" such as the following
     ///
     /// ```text
-    /// +-------+,
-    /// | count |,
-    /// +-------+,
-    /// | 6     |,
-    /// +-------+,
+    /// +-------+
+    /// | count |
+    /// +-------+
+    /// | 6     |
+    /// +-------+
     /// ```
     ///
     /// # See Also


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