Skip to content

Always set the copyProgress field.#32

Merged
edwin-huber merged 1 commit intoAzure:devfrom
trekawek:dev
Jul 14, 2018
Merged

Always set the copyProgress field.#32
edwin-huber merged 1 commit intoAzure:devfrom
trekawek:dev

Conversation

@trekawek
Copy link
Copy Markdown

@trekawek trekawek commented Jun 8, 2018

Before this change, the Azure Java SDK was failing on the list operation if it included copied blobs:

Exception in thread "main" java.util.NoSuchElementException: An error occurred while enumerating the result, check the original exception for details.
        at com.microsoft.azure.storage.core.LazySegmentedIterator.hasNext(LazySegmentedIterator.java:113)
[...]
Caused by: com.microsoft.azure.storage.StorageException: OK
        at com.microsoft.azure.storage.StorageException.translateException(StorageException.java:87)
        at com.microsoft.azure.storage.core.ExecutionEngine.executeWithRetry(ExecutionEngine.java:209)
        at com.microsoft.azure.storage.core.LazySegmentedIterator.hasNext(LazySegmentedIterator.java:109)
        ... 4 more
Caused by: java.lang.NumberFormatException: For input string: "undefined"
        at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
        at java.lang.Long.parseLong(Long.java:589)
        at java.lang.Long.parseLong(Long.java:631)
        at com.microsoft.azure.storage.blob.BlobListHandler.setProperties(BlobListHandler.java:312)
        at com.microsoft.azure.storage.blob.BlobListHandler.endElement(BlobListHandler.java:188)
[... SAX parsing ...]
        at com.microsoft.azure.storage.blob.BlobListHandler.getBlobList(BlobListHandler.java:75)
        at com.microsoft.azure.storage.blob.CloudBlobContainer$7.postProcessResponse(CloudBlobContainer.java:1473)
        at com.microsoft.azure.storage.blob.CloudBlobContainer$7.postProcessResponse(CloudBlobContainer.java:1437)
        at com.microsoft.azure.storage.core.ExecutionEngine.executeWithRetry(ExecutionEngine.java:155)

@edwin-huber
Copy link
Copy Markdown
Collaborator

Many thanks for your contribution!

@edwin-huber
Copy link
Copy Markdown
Collaborator

Travis failure OK, this is a flaky test for some reason, needs debugging to check how we get a race condition or similar, not a problem with this code.

@edwin-huber edwin-huber merged commit 7ef8615 into Azure:dev Jul 14, 2018
gpatryk pushed a commit to gpatryk/Azurite that referenced this pull request Dec 7, 2018
Always set the copyProgress field.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants