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.

Sign up for GitHub

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

Jump to bottom

"Cannot load CSV data with a nested schema" with a JSON file in load_data_by_post #129

Closed
Falydoor opened this issue Oct 16, 2015 · 3 comments · Fixed by #130
Closed

"Cannot load CSV data with a nested schema" with a JSON file in load_data_by_post #129

Falydoor opened this issue Oct 16, 2015 · 3 comments · Fixed by #130
Assignees
@jerjou
Labels
🚨 This issue needs some love. triage me I really want to be triaged.

Comments

@Falydoor
Copy link

Hello

With the new load_data_by_post, I'm not able to upload a JSON file and I have this error "Cannot load CSV data with a nested schema".

Sounds like the job expect a CSV not a JSON file so do I need to do something special ?

And I tested on the web client with my data/schema and it works fine, here the schema and the data in case you wanna test.

Regards
@theacodes
Copy link
Contributor

@jerjou do you mind checking this one out?

@jerjou jerjou mentioned this issue Oct 16, 2015
Merged
@jerjou
Copy link
Contributor

jerjou commented Oct 16, 2015

Oops - overlooked json data uploads. Thanks for catching that! PR pending..

@jerjou
Copy link
Contributor

jerjou commented Oct 16, 2015

Okay - should be fixed. Let us know if you run into any issues!

@yoshi-automation yoshi-automation added 🚨 This issue needs some love. triage me I really want to be triaged. labels Apr 7, 2020
engelke pushed a commit that referenced this issue Nov 9, 2022
@gwhitehawk
chore: added code samples for etag usage (#129) (#130)
e12ad08 
Co-authored-by: Miroslava Sotakova <mirka@google.com>
dandhlee pushed a commit that referenced this issue Nov 10, 2022
@gwhitehawk @dandhlee
chore: added code samples for etag usage (#129) (#130)
e69c7b0 
Co-authored-by: Miroslava Sotakova <mirka@google.com>
donmccasland pushed a commit that referenced this issue Nov 11, 2022
@parthea
chore: migrate to owlbot (#129)
a2a3061
@donmccasland donmccasland mentioned this issue Nov 11, 2022
Merged
8 tasks
@telpirion telpirion mentioned this issue Nov 16, 2022
Merged
8 tasks
telpirion pushed a commit that referenced this issue Nov 16, 2022
@renovate-bot
chore(deps): update dependency google-cloud-vision to v2.3.0 (#129)
3ec0ae7
msampathkumar pushed a commit that referenced this issue Nov 16, 2022
@gcf-owl-bot
chore: new owl bot post processor docker image (#129)
6fa42cf 
gcr.io/repo-automation-bots/owlbot-python:latest@sha256:3c3a445b3ddc99ccd5d31edc4b4519729635d20693900db32c4f587ed51f7479
@msampathkumar msampathkumar mentioned this issue Nov 16, 2022
Merged
8 tasks
dandhlee pushed a commit that referenced this issue Nov 17, 2022
@gcf-owl-bot @dandhlee
chore: new owl bot post processor docker image (#129)
89278ce 
gcr.io/repo-automation-bots/owlbot-python:latest@sha256:3c3a445b3ddc99ccd5d31edc4b4519729635d20693900db32c4f587ed51f7479
arbrown pushed a commit that referenced this issue Nov 17, 2022
@telpirion
chore: remove unneeded Translation and NL references (#129)
066bac1 
* chore: remove unneeded Translation and NL references

* fix: tests
@arbrown arbrown mentioned this issue Nov 17, 2022
Merged
8 tasks
dandhlee pushed a commit that referenced this issue Nov 17, 2022
@telpirion @dandhlee
chore: remove unneeded Translation and NL references (#129)
7ab804d 
* chore: remove unneeded Translation and NL references

* fix: tests
dandhlee pushed a commit that referenced this issue Nov 18, 2022
@renovate-bot @dandhlee
chore(deps): update dependency google-cloud-vision to v2.3.0 (#129)
bcab071
dandhlee pushed a commit that referenced this issue Jan 26, 2023
@parthea @dandhlee
chore: migrate to owlbot (#129)
9027344
telpirion pushed a commit that referenced this issue Mar 13, 2023
@parthea @telpirion
chore: migrate to owlbot (#129)
222f143
dizcology pushed a commit that referenced this issue Sep 11, 2023
@yoshi-automation
chore: add license headers (#129)
bda1bee 
* changes without context

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.

* chore: Add license headers for python config files

Source-Author: Anthonios Partheniou <partheniou@google.com>
Source-Date: Tue Apr 6 11:32:03 2021 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 5b5bf6d519b2d658d9f2e483d9f6f3d0ba8ee6bc
Source-Link: googleapis/synthtool@5b5bf6d
@dizcology dizcology mentioned this issue Sep 11, 2023
Merged
9 tasks
leahecole pushed a commit that referenced this issue Sep 15, 2023
@yoshi-automation @leahecole
chore: add license headers (#129)
98f9472 
* changes without context

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.

* chore: Add license headers for python config files

Source-Author: Anthonios Partheniou <partheniou@google.com>
Source-Date: Tue Apr 6 11:32:03 2021 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 5b5bf6d519b2d658d9f2e483d9f6f3d0ba8ee6bc
Source-Link: googleapis/synthtool@5b5bf6d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerjou jerjou

Labels
🚨 This issue needs some love. triage me I really want to be triaged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Specify source format, so json uploads work. GoogleCloudPlatform/python-docs-samples
4 participants
@theacodes @jerjou @Falydoor @yoshi-automation