Skip to content
This repository was archived by the owner on Nov 11, 2022. It is now read-only.

Conversation

@peihe
Copy link
Contributor

@peihe peihe commented Apr 15, 2016

No description provided.

@peihe
Copy link
Contributor Author

peihe commented Apr 15, 2016

R: @dhalperi

* This {@link PTransform} is used by the {@link DataflowPipelineTranslator} as a way
* to provide the native definition of the BigQuery sink.
*/
private static class BatchBigQueryIONativeWrite extends PTransform<PCollection<TableRow>, PDone> {
Copy link
Contributor

@dhalperi dhalperi Apr 15, 2016

Choose a reason for hiding this comment

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

I think this replaces BigQueryIOTranslator for write? Remove those?

Copy link
Contributor

@dhalperi dhalperi Apr 15, 2016

Choose a reason for hiding this comment

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

(Here and in Beam)

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

@dhalperi
Copy link
Contributor

LGTM. Can you:

  • confirm that when you build from github, we get native io by default
  • confirm that when you build from github, the experiment successfully switches back to BigQueryIO.Write
  • squash, so we can merge via the PR?

@peihe
Copy link
Contributor Author

peihe commented Apr 15, 2016

Done

@peihe
Copy link
Contributor Author

peihe commented Apr 15, 2016

There is a bug in BigQuery Custom Sink:
the pipeline will fail if there is double '/' in the concatenated tempPath.

I think we want to merge this PR, and I will send out a fix for the tempPath.

@dhalperi
Copy link
Contributor

Awesome!

@dhalperi dhalperi merged commit 66b2dec into GoogleCloudPlatform:master Apr 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants