beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sammcveety <...@git.apache.org>
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:

    https://github.com/apache/incubator-beam/pull/1238

    [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
error.

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

    $ git pull https://github.com/sammcveety/incubator-beam sgmc/bq_vp

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

    https://github.com/apache/incubator-beam/pull/1238.patch

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 <sgmc@google.com>
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 <sgmc@google.com>
Date:   2016-12-06T05:24:20Z

    Minor fixes

commit 1ca49af0659fd32ffa06e4e5778e3e4f90d85be6
Author: Sam McVeety <sgmc@google.com>
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message