beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sammcveety <>
Subject [GitHub] incubator-beam pull request #1238: [BEAM-551] BigqueryIO.Read support for Va...
Date Tue, 06 Dec 2016 21:10:54 GMT
GitHub user sammcveety reopened a pull request:

    [BEAM-551] BigqueryIO.Read support for ValueProvider 

    R: @dhalperi 
    This is the serialization issue I was referencing, where I think the issue is that there
is a deferred translation from String -> TableRef -> String, causing a serialization

You can merge this pull request into a Git repository by running:

    $ git pull sgmc/bq_vp

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1238
commit ced1073597e3fbcc826f10a417e810803c7317ce
Author: Sam McVeety <>
Date:   2016-10-30T18:58:44Z

    Add BQ VP code
    BQ VP
    Fix most tests
    Make serializable
    Fix BQ tests
    Fix tests
    Update API
    Fix validation case
    Fix one more query reference

commit 2d05f107499ce94bb35f05a4c386625e66fe660c
Author: Sam McVeety <>
Date:   2016-12-06T05:24:20Z

    Minor fixes

commit 1ca49af0659fd32ffa06e4e5778e3e4f90d85be6
Author: Sam McVeety <>
Date:   2016-12-06T05:31:34Z

    Address serialization issue


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message