Skip to content

BEAM-1269: Update bigtable library dependency and add cached data pools.#2377

Closed
mdshalda wants to merge 1 commit intoapache:masterfrom
mdshalda:mshalda/beam-1269
Closed

BEAM-1269: Update bigtable library dependency and add cached data pools.#2377
mdshalda wants to merge 1 commit intoapache:masterfrom
mdshalda:mshalda/beam-1269

Conversation

@mdshalda
Copy link
Contributor

Hi @dhalperi,

Can you review these changes to add the new bigtable library and cached data pools?

@dhalperi
Copy link
Contributor

R: @dhalperi

@dhalperi
Copy link
Contributor

Hi Mark,
It looks like you were not quite on head and have some conflicts with the existing code. Can you git fetch latest master and rebase?

@mdshalda mdshalda force-pushed the mshalda/beam-1269 branch 3 times, most recently from f1cd9cb to 74c00df Compare March 30, 2017 21:54
@mdshalda
Copy link
Contributor Author

I think I fixed this now.

@asfbot
Copy link

asfbot commented Mar 30, 2017

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

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 74c00df on mdshalda:mshalda/beam-1269 into ** on apache:master**.

@asfbot
Copy link

asfbot commented Mar 30, 2017

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

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 74c00df on mdshalda:mshalda/beam-1269 into ** on apache:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 74c00df on mdshalda:mshalda/beam-1269 into ** on apache:master**.

@asfbot
Copy link

asfbot commented Mar 30, 2017

@asfbot
Copy link

asfbot commented Mar 30, 2017

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

@dhalperi
Copy link
Contributor

dhalperi commented Mar 31, 2017

My guess: Bigtable upgraded their transitive requirements on gRPC and Protobuf:

googleapis/java-bigtable-hbase@a227311#diff-600376dffeb79835ede4a0b285078036

mimic those in the root pom.xml, and ignore the guava downgrade for now.

@asfbot
Copy link

asfbot commented Mar 31, 2017

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

Build result: FAILURE

[...truncated 2.11 MB...] at java.lang.Thread.run(Thread.java:745)Caused by: org.apache.maven.plugin.MojoExecutionException: Command execution failed. at org.codehaus.mojo.exec.ExecMojo.execute(ExecMojo.java:302) at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:134) at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208) ... 31 moreCaused by: org.apache.commons.exec.ExecuteException: Process exited with an error: 1 (Exit value: 1) at org.apache.commons.exec.DefaultExecutor.executeInternal(DefaultExecutor.java:404) at org.apache.commons.exec.DefaultExecutor.execute(DefaultExecutor.java:166) at org.codehaus.mojo.exec.ExecMojo.executeCommandLine(ExecMojo.java:764) at org.codehaus.mojo.exec.ExecMojo.executeCommandLine(ExecMojo.java:711) at org.codehaus.mojo.exec.ExecMojo.execute(ExecMojo.java:289) ... 33 more2017-03-31T16:24:37.150 [ERROR] 2017-03-31T16:24:37.150 [ERROR] Re-run Maven using the -X switch to enable full debug logging.2017-03-31T16:24:37.150 [ERROR] 2017-03-31T16:24:37.150 [ERROR] For more information about the errors and possible solutions, please read the following articles:2017-03-31T16:24:37.150 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException2017-03-31T16:24:37.150 [ERROR] 2017-03-31T16:24:37.150 [ERROR] After correcting the problems, you can resume the build with the command2017-03-31T16:24:37.150 [ERROR] mvn -rf :beam-sdks-pythonchannel stoppedSetting status of 8a007bb to FAILURE with url https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9000/ and message: 'Build finished. 'Using context: Jenkins: Maven clean install
--none--

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 70.324% when pulling e09f88c on mdshalda:mshalda/beam-1269 into 68522aa on apache:master.

@dhalperi
Copy link
Contributor

Kicked off https://builds.apache.org/view/Beam/job/beam_PostCommit_Java_MavenInstall/3111/ to see if the test passes

@asfbot
Copy link

asfbot commented Mar 31, 2017

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

@mdshalda mdshalda force-pushed the mshalda/beam-1269 branch from e09f88c to 973a8c9 Compare April 4, 2017 21:57
@mdshalda
Copy link
Contributor Author

mdshalda commented Apr 4, 2017

@dhalperi - Made the adjustments to the dependencies as discussed

@dhalperi
Copy link
Contributor

dhalperi commented Apr 4, 2017

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 70.017% when pulling 973a8c9 on mdshalda:mshalda/beam-1269 into 65ffd6c on apache:master.

@asfbot
Copy link

asfbot commented Apr 4, 2017

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

…ls for efficiency and remove pegging data channel count to 1.
@mdshalda mdshalda force-pushed the mshalda/beam-1269 branch from 973a8c9 to 2a7d9eb Compare April 5, 2017 21:55
@dhalperi
Copy link
Contributor

dhalperi commented Apr 5, 2017

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 69.833% when pulling 2a7d9eb on mdshalda:mshalda/beam-1269 into 9fb4fc3 on apache:master.

@asfbot
Copy link

asfbot commented Apr 5, 2017

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

@dhalperi
Copy link
Contributor

dhalperi commented Apr 5, 2017

Postcommit passed. Merging, thanks @mdshalda !

@asfgit asfgit closed this in 697b19f Apr 5, 2017
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.

4 participants

Comments