Skip to content

feat(bindings/c): add opendal_operator_blocking_delete method#2416

Merged
xyjixyjixyji merged 1 commit intoapache:mainfrom
jiaoew1991:c-blocking-delete
Jun 5, 2023
Merged

feat(bindings/c): add opendal_operator_blocking_delete method#2416
xyjixyjixyji merged 1 commit intoapache:mainfrom
jiaoew1991:c-blocking-delete

Conversation

@jiaoew1991
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Enwei Jiao <enwei.jiao@zilliz.com>
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jun 5, 2023
@Xuanwo Xuanwo requested a review from xyjixyjixyji June 5, 2023 08:39
@jiaoew1991 jiaoew1991 force-pushed the c-blocking-delete branch from c240128 to 99389fe Compare June 5, 2023 08:48
Copy link
Copy Markdown
Contributor

@xyjixyjixyji xyjixyjixyji left a comment

Choose a reason for hiding this comment

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

Thanks for the work, the function implementation is basically correct. But there exists a few points that need some polishments 😃

Comment thread bindings/c/src/lib.rs Outdated
Comment thread bindings/c/src/lib.rs
Comment thread bindings/c/tests/bdd.cpp
Comment thread bindings/c/src/lib.rs
@jiaoew1991 jiaoew1991 force-pushed the c-blocking-delete branch 2 times, most recently from c4cdefa to 03e018d Compare June 5, 2023 09:47
Copy link
Copy Markdown
Contributor

@xyjixyjixyji xyjixyjixyji left a comment

Choose a reason for hiding this comment

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

Thanks for the work! Lets wait the CI to pass.

@xyjixyjixyji xyjixyjixyji merged commit c3868eb into apache:main Jun 5, 2023
@suyanhanx suyanhanx mentioned this pull request Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants