Re: [PR] feat(bin/oli): support cp to dir [opendal]

2025-05-11 Thread via GitHub
xxchan commented on code in PR #6140: URL: https://github.com/apache/opendal/pull/6140#discussion_r2083900794 ## bin/oli/src/commands/cp.rs: ## @@ -84,28 +124,67 @@ impl CopyCmd { return Ok(()); } -let dst_root = Path::new(&dst_path); +//

Re: [PR] feat(bin/oli): support cp to dir [opendal]

2025-05-11 Thread via GitHub
Copilot commented on code in PR #6140: URL: https://github.com/apache/opendal/pull/6140#discussion_r2083893148 ## .github/workflows/ci_bin_oli.yml: ## @@ -45,7 +45,13 @@ jobs: need-rocksdb: true need-protoc: true github-token: ${{ secrets.GITHUB_

Re: [PR] refactor(bindings/java): AsyncOperator deprecate append methods in favor of write with WriteOptions where append = true (#6163) [opendal]

2025-05-11 Thread via GitHub
kingsword09 commented on PR #6169: URL: https://github.com/apache/opendal/pull/6169#issuecomment-2870935255 > Great! > > Generally looks good. IIRC you can do the same for the (blocking) Operator? I checked the (blocking) Operator and it seems to have no standalone append metho

[PR] feat(core): share one redb database between different Operators [opendal]

2025-05-11 Thread via GitHub
TD-Sky opened a new pull request, #6173: URL: https://github.com/apache/opendal/pull/6173 # Which issue does this PR close? Closes #6172 . # What changes are included in this PR? - [x] `RedbBuilder::database(self, db: &Arc) -> Self` - [x] Creating the specified table a

[I] new feature: support multi-tables access for redb [opendal]

2025-05-11 Thread via GitHub
TD-Sky opened a new issue, #6172: URL: https://github.com/apache/opendal/issues/6172 ### Feature Description Allow to share one redb database connection between different Operators. ### Problem and Solution The redb database connection is exclusive, which means opening sa

Re: [PR] feat(bindings/python): Enhance Reader and Writer [opendal]

2025-05-11 Thread via GitHub
chitralverma commented on PR #6086: URL: https://github.com/apache/opendal/pull/6086#issuecomment-2870801659 @messense @Xuanwo added some benchmarks and updated the scope of this PR. See [Comment](https://github.com/apache/opendal/issues/5943#issuecomment-2858120315) Please l

Re: [PR] refactor(bindings/java): AsyncOperator deprecate append methods in favor of write with WriteOptions where append = true (#6163) [opendal]

2025-05-11 Thread via GitHub
kingsword09 commented on PR #6169: URL: https://github.com/apache/opendal/pull/6169#issuecomment-2870530883 > Thanks for your contribution! > > I'd suggest you remove the rust part code also, and actually you can remove these Java method as well with proper upgrade.md changelog.

[I] new feature: Implement raw::Access for ObjectStore in object_store_opendal [opendal]

2025-05-11 Thread via GitHub
flaneur2020 opened a new issue, #6171: URL: https://github.com/apache/opendal/issues/6171 ### Feature Description related: https://github.com/apache/opendal/discussions/6162#discussioncomment-13108413 ### Problem and Solution > Following the suggestions from @flaneur2020

Re: [I] new feature: Implement raw::Access for ObjectStore in object_store_opendal [opendal]

2025-05-11 Thread via GitHub
flaneur2020 commented on issue #6171: URL: https://github.com/apache/opendal/issues/6171#issuecomment-2870513371 take this :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[PR] dev: extract service config structs' identifier [opendal]

2025-05-11 Thread via GitHub
erickguan opened a new pull request, #6170: URL: https://github.com/apache/opendal/pull/6170 # Which issue does this PR close? Originated from https://github.com/apache/opendal/pull/5457#discussion_r1897695599. Also for generating service pages in bindings. e.g., Python. Something

(opendal) 01/01: dev: extract service config structs' identifier

2025-05-11 Thread erickguan
This is an automated email from the ASF dual-hosted git repository. erickguan pushed a commit to branch docs-extraction in repository https://gitbox.apache.org/repos/asf/opendal.git commit e898f3780685a29ea4ee0ca99509e013e7eaf571 Author: Erick Guan <297343+erickg...@users.noreply.github.com> Auth

(opendal) branch docs-extraction created (now e898f3780)

2025-05-11 Thread erickguan
This is an automated email from the ASF dual-hosted git repository. erickguan pushed a change to branch docs-extraction in repository https://gitbox.apache.org/repos/asf/opendal.git at e898f3780 dev: extract service config structs' identifier This branch includes the following new commits

Re: [PR] feat(bin/oli): support cp to dir [opendal]

2025-05-11 Thread via GitHub
asukaminato0721 commented on code in PR #6140: URL: https://github.com/apache/opendal/pull/6140#discussion_r2083477308 ## .github/workflows/ci_bin_oli.yml: ## @@ -46,6 +46,20 @@ jobs: need-protoc: true github-token: ${{ secrets.GITHUB_TOKEN }} + - us

Re: [PR] feat(bin/oli): support cp to dir [opendal]

2025-05-11 Thread via GitHub
asukaminato0721 commented on code in PR #6140: URL: https://github.com/apache/opendal/pull/6140#discussion_r2083475949 ## .github/workflows/ci_bin_oli.yml: ## @@ -46,6 +46,20 @@ jobs: need-protoc: true github-token: ${{ secrets.GITHUB_TOKEN }} + - us

Re: [PR] feat(bin/oli): support cp to dir [opendal]

2025-05-11 Thread via GitHub
asukaminato0721 commented on code in PR #6140: URL: https://github.com/apache/opendal/pull/6140#discussion_r2083470469 ## .github/workflows/ci_bin_oli.yml: ## @@ -46,6 +46,20 @@ jobs: need-protoc: true github-token: ${{ secrets.GITHUB_TOKEN }} + - us