-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
storage: PCU hangs #2515
Comments
is this being run from a Google Compute Engine instance? |
Yes, I ran the test from a GCE VM. |
I have tried your same code with smaller file sizes and it has completed just fine for me. The 4GB file is "hanging" but I think this is just file generation + upload time coming from my laptop. Working on running it from GCE. Have you experienced this for other file sizes? |
as for the 16mib intermediate objects that is correct as the per worker buffer size is set by transfer manager (of which 16 mib is the default) so that is to be expected. There will be intermediate composes to reach the final object. |
I ran the following on my personal GCE instance with success. |
perhaps if we got a thread dump we could assist you better |
I will try publishing my reproducer more completely: https://github.com/benjaminp/java-storage-puc-hang I put a bucket name in the
This last command hangs. The stacks are not too illuminating to me anyway:
|
We were able to reproduce this finally. We don't run with heaps that small typically which is why we were unable to reproduce it initially. Turns out we were suppressing an OOM causing the operation to hang. |
fix went out in #2533 closing. |
Environment details
General, Core, and Other are also allowed as types
Steps to reproduce
I wanted to try out the new parallel composite support, but my example below simply hangs. The program completes with
.setAllowParallelCompositeUpload(false)
.I see it created many 16mib intermediate objects in the bucket, though. (That also seems wrong since 4gib/16mib ≫32, the maximum number of composite object components.)
Code example
The text was updated successfully, but these errors were encountered: