Skip to content

Remove the examples/cli.rs interactive CLI example - #108

Draft
alamb wants to merge 3 commits into
mainfrom
alamb/remove-cli-example
Draft

alamb wants to merge 3 commits into
mainfrom
alamb/remove-cli-example

Conversation

@alamb

@alamb alamb commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

I am adding an easier to use interactive shell, dfv (a datafusion-cli with the Variant functions preinstalled), in #102. The hand-rolled examples/cli.rs REPL duplicates that and is less capable, so this PR removes it.

What changes are included in this PR?

Are these changes tested?

cargo check --examples --tests passes. No functional code is changed.

Are there any user-facing changes?

cargo run --example cli no longer exists. Use dfv from #102 instead.

The interactive CLI example is superseded by dfv (#102), a datafusion-cli
with the Variant functions preinstalled. Drop the example, its rustyline
dev-dependency, and the README instructions for it.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Comment thread examples/cli.rs
#[tokio::main]
async fn main() -> Result<()> {
// todo: we can let users pass in arbitrary files to load
// for now, we'll directly use bluesky's data

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this feature is implemented in #102

Comment thread README.md

## More examples and benchmarks

The [interactive CLI](https://github.com/datafusion-contrib/datafusion-variant/blob/main/examples/cli.rs)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will recover the bsky data here: #109

alamb and others added 2 commits October 28, 2026 10:30
The jsonbench prepare command downloads the same Bluesky files itself, so
the script is redundant. Point profile_parse_json at that command instead.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant