Skip to content

[BEAM-2076] Shade Additional Direct Runner Dependencies#2973

Closed
tgroh wants to merge 1 commit intoapache:masterfrom
tgroh:direct_runner_deps_less_core
Closed

[BEAM-2076] Shade Additional Direct Runner Dependencies#2973
tgroh wants to merge 1 commit intoapache:masterfrom
tgroh:direct_runner_deps_less_core

Conversation

@tgroh
Copy link
Member

@tgroh tgroh commented May 8, 2017

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify.
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Add a DirectRunner API Surface Test.

Repackaging runners-core appears to cause the direct runner to hang.
Investigations in progress.

Add a DirectRunner API Surface Test.

Repackaging runners-core appears to cause the direct runner to hang.
Investigations in progress.
@tgroh
Copy link
Member Author

tgroh commented May 8, 2017

R: @dhalperi

@dhalperi
Copy link
Contributor

dhalperi commented May 8, 2017

LGTM assuming green. Per our discussion, may still be some issues with protobuf if runners-core uses it.

@tgroh
Copy link
Member Author

tgroh commented May 9, 2017

Preempted by complete fix in #2714

@tgroh tgroh closed this May 9, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 70.591% when pulling 6ce1631 on tgroh:direct_runner_deps_less_core into 152c5bc on apache:master.

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.

3 participants