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

Conversation

@nevillelyh
Copy link
Contributor

No description provided.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@dhalperi dhalperi self-assigned this Mar 23, 2016
@dhalperi
Copy link
Contributor

Hah! Thanks, Neville for catching this!

Some news -- now that Dataflow SDK is morphing into Apache Beam (incubating), we're going to stop taking new pull requests on this repo. There will be a "contribution guide" up at that page sometime soon, and for now you can see a bunch of resources (mailing lists, issue tracker, etc.) under the "Community" tab on that website.

Of course, I will happily go ahead and port this change over to the Beam repo and make sure it ends up here as well. Expect to see this auto-closed by a git merge soon.

Thanks again, as always, Neville!

@dhalperi
Copy link
Contributor

CLA: confirmed via Spotify

@dhalperi dhalperi added cla: yes and removed cla: no labels Mar 23, 2016
@dhalperi
Copy link
Contributor

Note this is now a backport of apache/beam#68

@dhalperi dhalperi changed the title make BigtableIO#Read#withRowFilter public backport apache/incubator-beam#68 Mar 23, 2016
@dhalperi dhalperi merged commit be14afa into GoogleCloudPlatform:master Mar 23, 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.

4 participants