Skip to content

[BEAM-1349] Merge python-sdk to master#1872

Merged
asfgit merged 937 commits intomasterfrom
python-sdk
Jan 30, 2017
Merged

[BEAM-1349] Merge python-sdk to master#1872
asfgit merged 937 commits intomasterfrom
python-sdk

Conversation

@aaltay
Copy link
Member

@aaltay aaltay commented Jan 30, 2017

No description provided.

mariapython and others added 30 commits November 22, 2016 09:57
Convert StarterPipeline ParDo to MapElements.

Use the new DoFn for non-outputting transforms.
1. setCurrentContainer returns the previous MetricsEnvironment
2. setCurrentContainer(null) resets the thread local
3. scopedCurrentContainer sets the container and returns a Closeable to
   reset the previous container.
This allows reuse of keys for downstream serialization.
This tests basic features of TransformHierarchy
update datastore_wordcount example to include writes
This is roughly equivalent to "RunnableOnService" in the Java SDK. See
BEAM-655
  - pin googledatastore version to 6.4.1
  - add num_shards options to datastore wordcount example
  - move datastore wordcount example to cookbook directory
aaltay and others added 24 commits January 25, 2017 16:16
test_with_requirements_file

The test fails if there are leftover files in the default folder for
requirements cache either from earlier tests, or from the previous
workspaces.
In order to use counter structured names, and add new metrics (e.g.
Distributions), we need to update the dataflow client protocol buffers.
Moves ThreadPool creation to a util function.
Records and resets logging level due to this being reset by  apitools when used with a ThreadPool.
Unit tests, testing retries does not need to use real time. This change
reduces the total tox time for unit tests from 235 seconds to 73 seconds
locally.
@aaltay
Copy link
Member Author

aaltay commented Jan 30, 2017

R: @davorbonaci

@asfbot
Copy link

asfbot commented Jan 30, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/6896/
--none--

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 69.62% when pulling be0e32e on python-sdk into 847e4e9 on master.

@davorbonaci
Copy link
Member

Comments:

  • Files in runners/ are probably faulty merges. Investigate and probably revert?
  • Move sdks/python/.gitignore to top-level .gitignore?
  • Remove sdks/python/LICENSE?

Didn't look carefully inside sdks/python, but I suspect there should be some updates to Jenkins configuration and elsewhere too. I'd be fine moving forward and fixing it in a follow-up.

@asfgit asfgit merged commit be0e32e into master Jan 30, 2017
@asfgit asfgit deleted the python-sdk branch January 31, 2017 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.