You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Luke Cwik (JIRA)" <ji...@apache.org> on 2018/01/12 18:18:00 UTC
[jira] [Comment Edited] (BEAM-3250) Migrate ValidatesRunner Jenkins
PostCommits to Gradle
[ https://issues.apache.org/jira/browse/BEAM-3250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16324326#comment-16324326 ]
Luke Cwik edited comment on BEAM-3250 at 1/12/18 6:17 PM:
----------------------------------------------------------
I worked with Ben Sidhom on getting Flink ValidatesRunner to work within the Flink project:
https://github.com/bsidhom/beam/tree/flink-validate-runner
But it was a mess because we couldn't figure out a good way to get the dependenciesToScan like option to work in Gradle test tasks.
The next best thing I suggested was to do what I did within examples/java/build.gradle and make all the runners have their validates runner post commit definition within sdks/java/core/build.gradle
was (Author: lcwik):
I worked with Ben on getting Flink ValidatesRunner to work within the Flink project:
https://github.com/bsidhom/beam/tree/flink-validate-runner
But it was a mess because we couldn't figure out a good way to get the dependenciesToScan like option to work in Gradle test tasks.
The next best thing I suggested was to do what I did within examples/java/build.gradle and make all the runners have their validates runner post commit definition within sdks/java/core/build.gradle
> Migrate ValidatesRunner Jenkins PostCommits to Gradle
> -----------------------------------------------------
>
> Key: BEAM-3250
> URL: https://issues.apache.org/jira/browse/BEAM-3250
> Project: Beam
> Issue Type: Sub-task
> Components: build-system, testing
> Reporter: Luke Cwik
> Assignee: Kenneth Knowles
>
> Update these targets to execute ValidatesRunner tests: https://github.com/apache/beam/search?l=Groovy&q=ValidatesRunner&type=&utf8=%E2%9C%93
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)