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