You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "ClSlaid (via GitHub)" <gi...@apache.org> on 2023/04/06 12:29:27 UTC

[GitHub] [incubator-opendal] ClSlaid opened a new pull request, #1863: feat: add move support for opendal

ClSlaid opened a new pull request, #1863:
URL: https://github.com/apache/incubator-opendal/pull/1863

   `move` is on the 2023 road map of opendal. This PR implements this feature.
   
   == implemented:
   
   1. raw::accessor api's blocking `moves` and `blocking_moves`
   2. Operator's `moves`
   3. BlockingOperator's `blocking_moves`
   4. services::fs's `moves` and `blocking_moves`
   5. behavior test's `moves` and `blocking_moves` tests
   
   == notes about this PR:
   
   - `move` unfortunately is a keyword in rust, so I have to use `moves`
   - thanks to @suyanhanxx's copy support


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-opendal] ClSlaid commented on pull request #1863: feat: add move support for opendal

Posted by "ClSlaid (via GitHub)" <gi...@apache.org>.
ClSlaid commented on PR #1863:
URL: https://github.com/apache/incubator-opendal/pull/1863#issuecomment-1498997651

   oh, just hours ago #1862 is merged 🥇
   
   I'll change this PR for it, let's convert this PR to draft now.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-opendal] suyanhanx commented on pull request #1863: feat: add move support for opendal

Posted by "suyanhanx (via GitHub)" <gi...@apache.org>.
suyanhanx commented on PR #1863:
URL: https://github.com/apache/incubator-opendal/pull/1863#issuecomment-1499088608

   I see you still wrote more than I did. It would be great if you could add them. Thank you.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-opendal] ClSlaid commented on pull request #1863: feat: add move support for opendal

Posted by "ClSlaid (via GitHub)" <gi...@apache.org>.
ClSlaid commented on PR #1863:
URL: https://github.com/apache/incubator-opendal/pull/1863#issuecomment-1499001525

   closed for implemented.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-opendal] ClSlaid closed pull request #1863: feat: add move support for opendal

Posted by "ClSlaid (via GitHub)" <gi...@apache.org>.
ClSlaid closed pull request #1863: feat: add move support for opendal
URL: https://github.com/apache/incubator-opendal/pull/1863


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org