You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by lc...@apache.org on 2018/01/24 21:38:15 UTC

[beam] branch master updated (12e30a3 -> e39964c)

This is an automated email from the ASF dual-hosted git repository.

lcwik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git.


    from 12e30a3  Merge pull request #4473
     add 6389f38  [BEAM-3412] Upgrade Cloud Bigtable to 1.0.0 The 1.0.0 Cloud Bigtable client upgrade included a non-backwards compatible change in protobuf generation.  Specifically, addAllXXXX() methods no longer work correctly.  `BigtableServiceImpl` used `addAllMutations()`, which caused run-time breakages.  To get around that, add each Mutation independently to the protobuf object. This PR also adds unit tests for BigtableServiceImpl so that read/write interactions do not cause runti [...]
     add 1559e37  Merge remote-tracking branch 'upstream/master' into upgrade_cbt_100
     add 0372db0  BigtableServiceImplTest now uses a List of ranges. The PR did not previously compile, since I had a pretty old version of the code in my development environment.  There was a non-backwards compatible change that I had to fix.
     add 42868b5  Updating Cloud Bigtable dependnecy to 1.0.0 in build.gradle.
     add 1095ac5  Using 1.0.0-pre3 for bigtable-proto. This should allow Cloud Bigtable's client to upgrade to 1.0.0 while keeping backwards compatibility with Beam's protobuf version.
     add c0da25c  Merge branch 'master' into upgrade_cbt_100
     add bd8d075  Updating build.gradle with a bigtable_proto_version of 1.0.0-pre3
     add 98d32ad  Fixing a bad merge in BigtableServiceImpl.
     new e39964c  Merge pull request #4462 from sduskis/upgrade_cbt_100

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 build.gradle                                       |   5 +-
 pom.xml                                            |   5 +-
 .../sdk/io/gcp/bigtable/BigtableServiceImpl.java   |  41 +++---
 .../io/gcp/bigtable/BigtableServiceImplTest.java   | 146 +++++++++++++++++++++
 4 files changed, 171 insertions(+), 26 deletions(-)
 create mode 100644 sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/bigtable/BigtableServiceImplTest.java

-- 
To stop receiving notification emails like this one, please contact
lcwik@apache.org.

[beam] 01/01: Merge pull request #4462 from sduskis/upgrade_cbt_100

Posted by lc...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

lcwik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git

commit e39964ca283adc6336b6ea8fda4aee74fa442b1d
Merge: 12e30a3 98d32ad
Author: Lukasz Cwik <lc...@google.com>
AuthorDate: Wed Jan 24 13:38:10 2018 -0800

    Merge pull request #4462 from sduskis/upgrade_cbt_100
    
    [BEAM-3412] Upgrade Cloud Bigtable to 1.0.0

 build.gradle                                       |   5 +-
 pom.xml                                            |   5 +-
 .../sdk/io/gcp/bigtable/BigtableServiceImpl.java   |  41 +++---
 .../io/gcp/bigtable/BigtableServiceImplTest.java   | 146 +++++++++++++++++++++
 4 files changed, 171 insertions(+), 26 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
lcwik@apache.org.