You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "Xuanwo (via GitHub)" <gi...@apache.org> on 2024/01/09 13:25:18 UTC

[PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   Fix https://github.com/apache/incubator-opendal/issues/3956


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   > LGTM. Let's add some tests in the future to ensure the retry behaviors.
   
   Yep, I did found more problems..


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #3958:
URL: https://github.com/apache/incubator-opendal/pull/3958


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   > It seems our google driver test key has been revoked. LOL
   
   Yep, it's expired. Not related to this PR.


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   LGTM. Let's add some tests to ensure the retry behaviors in the future.


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   GitHub issue caused CI failure.


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   cc @WenyXu for review.


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


Re: [PR] feat(core): Allow retry in concurrent write operations [incubator-opendal]

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

   It seems our google driver test key has been revoked. LOL 
   ```
      "error": "invalid_grant",
     "error_description": "Token has been expired or revoked."
   ```


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