Skip to content
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

Wrong json model on Data Lake Storage "Path" #16401

Closed
asafg6 opened this issue Dec 6, 2021 · 8 comments
Closed

Wrong json model on Data Lake Storage "Path" #16401

asafg6 opened this issue Dec 6, 2021 · 8 comments
Assignees
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Data Lake Storage Gen2 Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Previous Versions Work related to track1 and track1.5 SDKs question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@asafg6
Copy link
Contributor

asafg6 commented Dec 6, 2021

Bug Report

  • import path of package in question: github.com/Azure/azure-sdk-for-go/services/storage/datalake/2019-10-31/storagedatalake
  • SDK version: 58.3.0
  • output of go version: go version go1.17 linux/amd64
  • What happened?
    While trying to list paths got json parse error
Error = 'json: cannot unmarshal string into Go struct field Path.paths.contentLength of type int64' JSON = '{"paths":[{"contentLength":"0","etag":"0x8D9B8C1E77A2CCF","group":"$superuser","isDirectory":"true","lastModified":"Mon, 06 Dec 2021 14:08:40 GMT","name":"vms/7e5ffad1-c24d-11e8-911f-c4b301c47c6b","owner":"$superuser","permissions":"rwxr-x---"},{"contentLength":"0","etag":"0x8D9B8C4BB9B08F6","group":"$superuser","isDirectory":"true","lastModified":"Mon, 06 Dec 2021 14:28:55 GMT","name":"vms/dd981f75-c6ed-11e8-bb55-c4b301c47c6b","owner":"$superuser","permissions":"rwxr-x---"},{"contentLength":"24","etag":"0x8D9B8CD715E87DE","group":"$superuser","lastModified":"Mon, 06 Dec 2021 15:31:15 GMT","name":"vms/last","owner":"$superuser","permissions":"rw-r-----"}]}
  • What did you expect or want to happen?
    To get the list of paths from the api
  • How can we reproduce it?
    Use PathClient.List
  • Anything we should know about your environment.
    Using linux and go 1.17, nothing special
@ghost ghost added needs-triage This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Dec 6, 2021
asafg6 added a commit to veertuinc/azure-sdk-for-go that referenced this issue Dec 6, 2021
@RickWinter RickWinter added Data Lake Storage Gen2 Mgmt This issue is related to a management-plane library. Previous Versions Work related to track1 and track1.5 SDKs labels Dec 6, 2021
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Dec 6, 2021
@RickWinter RickWinter assigned zezha-msft and lirenhe and unassigned zezha-msft Dec 6, 2021
@asafg6
Copy link
Contributor Author

asafg6 commented Dec 7, 2021

I have this pull request that solves this issue
#16403

@ArcturusZhang
Copy link
Member

ArcturusZhang commented Dec 15, 2021

Hi @asafg6 thank you for your issue!

This is kind of special case. The package you are using belongs to data-plane, we stopped auto-generate SDK for dataplane swaggers for quite a time, therefore as a work around, we could merge your PR as a fix. But we recommend you switch to use official data plane SDK.
Let me confirm with someone to see if those are ready now...

@ArcturusZhang ArcturusZhang removed the Mgmt This issue is related to a management-plane library. label Dec 15, 2021
@ArcturusZhang
Copy link
Member

Hey @asafg6 for normal cases we have cut the new features for data plane packages now, but considering your change is a bug fix, and there is really no alternative of this package out there now, we will have a review and merge your PR to unblock you.
Sorry for the inconvenience

@asafg6
Copy link
Contributor Author

asafg6 commented Dec 20, 2021

Thank you @ArcturusZhang
What is the official data plane sdk?
I thought that this is it..

@lirenhe
Copy link

lirenhe commented Dec 20, 2021

@asafg6, here is the list of client libraries for GO: https://azure.github.io/azure-sdk/releases/latest/go.html

@asafg6
Copy link
Contributor Author

asafg6 commented Dec 20, 2021

@lirenhe Which one of those is the recommended sdk for Data Lake ???

@lirenhe
Copy link

lirenhe commented Dec 20, 2021

There is no data lake now, that is why @ArcturusZhang mentioned, 'there is really no alternative of this package out there now'.
We will have a review and merge your PR to unblock you.

@asafg6
Copy link
Contributor Author

asafg6 commented Dec 20, 2021

Ok thanks :) !

@RickWinter RickWinter added the Mgmt This issue is related to a management-plane library. label Jan 14, 2022
@ghost ghost added the needs-team-attention This issue needs attention from Azure service team or SDK team label Jan 14, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Data Lake Storage Gen2 Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Previous Versions Work related to track1 and track1.5 SDKs question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

5 participants