-
Notifications
You must be signed in to change notification settings - Fork 18
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
ShareUsageBytes is int32 #28
Comments
Hi @djyou, thanks for reaching out and letting us know this problem! That code was generated. I see it should be modified to 64 bit integer type to cover the right range of numbers. |
Because I lack write access to this repo, I can't milestone this issue |
@adreed-msft you have it now 😄 |
@mohsha-msft there's a PR here: #29 |
@adreed-msft Here is the link of build where I noticed one test case failing |
Hi @zezha-msft , @adreed-msft and @mohsha-msft , It seems like this issue is still present since the change made by @adreed-msft (#29) was reverted by @mohsha-msft (#40) because of failing tests (https://travis-ci.org/github/Azure/azure-storage-file-go/builds/724857628). Do you plan on providing a new fix for this issue? |
Why is int32 used to represent
ShareUsageBytes
?azure-storage-file-go/azfile/zz_generated_models.go
Line 2946 in 825ad79
Which version of the SDK was used?
Which platform are you using? (ex: Windows, Linux, Debian)
What problem was encountered?
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
The text was updated successfully, but these errors were encountered: