Skip to content

Conversation

@dhalperi
Copy link
Contributor

R: @peihe

// Require a schema if creating a table.
checkArgument(
createDisposition != CreateDisposition.CREATE_IF_NEEDED || jsonSchema != null,
"CreateDisposition is %s, however no schema was provided.",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to use %s, it only fails if CreateDisposition is CREATE_IF_NEEDED

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@peihe
Copy link
Contributor

peihe commented Apr 19, 2016

LGTM

@dhalperi dhalperi force-pushed the bigquery-write-project-id branch from 8419e88 to f61d4e2 Compare April 19, 2016 00:40
@dhalperi dhalperi force-pushed the bigquery-write-project-id branch from f61d4e2 to e3c876c Compare April 19, 2016 00:44
@asfgit asfgit merged commit e3c876c into apache:master Apr 19, 2016
asfgit pushed a commit that referenced this pull request Apr 19, 2016
@dhalperi dhalperi deleted the bigquery-write-project-id branch April 19, 2016 01:51
swegner pushed a commit to swegner/beam that referenced this pull request Apr 22, 2016
Override BigQueryIO.Write with native sink for Dataflow runner
peihe pushed a commit to peihe/DataflowJavaSDK that referenced this pull request Apr 26, 2016
iemejia added a commit to iemejia/beam that referenced this pull request Jan 12, 2018
mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018
pl04351820 pushed a commit to pl04351820/beam that referenced this pull request Dec 20, 2023
Leaving existing (old) conftest JSON files in place for now. in order
to  get the conftest runner working using existing semantics,
before updating the JSON files and making required changes (to use
'update_transforms', for instance) in a future PR.

Closes apache#95.

Co-authored-by: Christopher Wilcox <crwilcox@google.com>
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.

3 participants