[BEAM-59] Beam GcsFileSystem: implementation of match(). #2022
Conversation
|
Refer to this link for build results (access rights to CI server needed): Build result: FAILURE[...truncated 5406 lines...] at hudson.remoting.UserRequest.perform(UserRequest.java:153) at hudson.remoting.UserRequest.perform(UserRequest.java:50) at hudson.remoting.Request$2.run(Request.java:332) at hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:68) at java.util.concurrent.FutureTask.run(FutureTask.java:266) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) at java.lang.Thread.run(Thread.java:745)Caused by: org.apache.maven.plugin.MojoFailureException: You have 1 Checkstyle violation. at org.apache.maven.plugin.checkstyle.CheckstyleViolationCheckMojo.execute(CheckstyleViolationCheckMojo.java:588) at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:134) at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208) ... 31 more2017-02-16T19:44:59.476 [ERROR] 2017-02-16T19:44:59.476 [ERROR] Re-run Maven using the -X switch to enable full debug logging.2017-02-16T19:44:59.476 [ERROR] 2017-02-16T19:44:59.476 [ERROR] For more information about the errors and possible solutions, please read the following articles:2017-02-16T19:44:59.476 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException2017-02-16T19:44:59.477 [ERROR] 2017-02-16T19:44:59.477 [ERROR] After correcting the problems, you can resume the build with the command2017-02-16T19:44:59.477 [ERROR] mvn -rf :beam-sdks-java-io-google-cloud-platformchannel stoppedSetting status of d88f8d6 to FAILURE with url https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/7482/ and message: 'Build finished. 'Using context: Jenkins: Maven clean install--none-- |
peihe
force-pushed
the
file-system-gs-match
branch
from
d88f8d6
to
abe5462
Compare
|
Refer to this link for build results (access rights to CI server needed): |
peihe
force-pushed
the
file-system-gs-match
branch
from
abe5462
to
68382aa
Compare
|
PTAL |
| return MatchResult.create(Status.ERROR, e); | ||
| } | ||
| }}) | ||
| .toList(); |
There was a problem hiding this comment.
I presume we will eventually want to BULK or at least parallelize this.
There was a problem hiding this comment.
Assuming I'm right, can you please file a JIRA and put a TODO here (and on MatchNonGlobs?) ?
|
Changes Unknown when pulling 2c9babb on peihe:file-system-gs-match into ** on apache:master**. |
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
|
Still LGTM, merged now. |
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
[BEAM-<Jira issue #>] Description of pull requestmvn clean verify. (Even better, enableTravis-CI on your fork and ensure the whole test matrix passes).
<Jira issue #>in the title with the actual Jira issuenumber, if there is one.
Individual Contributor License Agreement.
This PR is based on #2015