You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by GitBox <gi...@apache.org> on 2022/07/24 12:26:02 UTC
[GitHub] [jclouds] gaul opened a new pull request, #146: Synchronize ByteSourcePayload Closer methods
gaul opened a new pull request, #146:
URL: https://github.com/apache/jclouds/pull/146
`Closer` is not thread safe and lack of synchronization can throw
`ArrayIndexOutOfBoundsException`s. Fixes gaul/s3proxy#303.
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] gaul closed pull request #146: Synchronize ByteSourcePayload in LocalBlobStore
Posted by GitBox <gi...@apache.org>.
gaul closed pull request #146: Synchronize ByteSourcePayload in LocalBlobStore
URL: https://github.com/apache/jclouds/pull/146
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] gaul commented on pull request #146: Synchronize ByteSourcePayload in LocalBlobStore
Posted by GitBox <gi...@apache.org>.
gaul commented on PR #146:
URL: https://github.com/apache/jclouds/pull/146#issuecomment-1201139074
Superseded by #150.
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] nacx commented on pull request #146: Synchronize ByteSourcePayload Closer methods
Posted by GitBox <gi...@apache.org>.
nacx commented on PR #146:
URL: https://github.com/apache/jclouds/pull/146#issuecomment-1193770640
If the LocalBlobStore can sync ina. narrower scope that would be much better
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] gaul commented on pull request #146: Synchronize ByteSourcePayload in LocalBlobStore
Posted by GitBox <gi...@apache.org>.
gaul commented on PR #146:
URL: https://github.com/apache/jclouds/pull/146#issuecomment-1200124644
Something seems strange here -- every call to `getBlob` should return a new `Payload`...
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] gaul commented on pull request #146: Synchronize ByteSourcePayload in LocalBlobStore
Posted by GitBox <gi...@apache.org>.
gaul commented on PR #146:
URL: https://github.com/apache/jclouds/pull/146#issuecomment-1200116416
> If the LocalBlobStore can sync ina. narrower scope that would be much better
Done.
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jclouds] gaul commented on pull request #146: Synchronize ByteSourcePayload Closer methods
Posted by GitBox <gi...@apache.org>.
gaul commented on PR #146:
URL: https://github.com/apache/jclouds/pull/146#issuecomment-1193308375
I think there is a narrower fix where `LocalBlobStore` could synchronize its calls sites instead. @nacx what do you think?
--
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: notifications-unsubscribe@jclouds.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org