Skip to content

chore: remove unused dependencies from core/examples/basic#6730

Closed
cfanbo wants to merge 2 commits intoapache:mainfrom
cfanbo:fix/example_basic
Closed

chore: remove unused dependencies from core/examples/basic#6730
cfanbo wants to merge 2 commits intoapache:mainfrom
cfanbo:fix/example_basic

Conversation

@cfanbo
Copy link
Copy Markdown

@cfanbo cfanbo commented Oct 23, 2025

remove unused dependencies from core/examples/basic and update coments

@cfanbo cfanbo requested a review from Xuanwo as a code owner October 23, 2025 08:46
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell labels Oct 23, 2025

[dependencies]
futures = "0.3"
# futures = "0.3"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please just remove it if we don't need it.


async fn example(op: Operator) -> Result<()> {
// Write data to S3.
// Write data to fs.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This comment is not quite true since the storage could be any storage configured.

@cfanbo cfanbo closed this Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants