Merge pull request #210 from spotify/bqjob-success

narape · web-flow · commit 16f30bd0800a · 2019-11-12T15:49:40.000+01:00
Make success hook public
diff --git a/contrib/flo-bigquery/src/main/java/com/spotify/flo/contrib/bigquery/BigQueryOperation.java b/contrib/flo-bigquery/src/main/java/com/spotify/flo/contrib/bigquery/BigQueryOperation.java
@@ -52,7 +52,7 @@ BigQueryOperation<T> job(Fn<JobInfo> jobRequest) {
   /**
    * Specify some action to take on success. E.g. publishing a staging table.
    */
-  BigQueryOperation<T> success(F1<JobInfo, T> success) {
+  public BigQueryOperation<T> success(F1<JobInfo, T> success) {
     this.success = Objects.requireNonNull(success);
     return this;
   }

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ BigQueryOperation<T> job(Fn<JobInfo> jobRequest) {`
`52`	`52`	`/**`
`53`	`53`	`* Specify some action to take on success. E.g. publishing a staging table.`
`54`	`54`	`*/`
`55`		`- BigQueryOperation<T> success(F1<JobInfo, T> success) {`
	`55`	`+ public BigQueryOperation<T> success(F1<JobInfo, T> success) {`
`56`	`56`	`this.success = Objects.requireNonNull(success);`
`57`	`57`	`return this;`
`58`	`58`	`}`