Skip to content

Conversation

scala-steward
Copy link
Contributor

Updates

  • org.apache.beam:beam-runners-direct-java
  • org.apache.beam:beam-runners-google-cloud-dataflow-java
  • org.apache.beam:beam-sdks-java-core
  • org.apache.beam:beam-sdks-java-extensions-sql
  • org.apache.beam:beam-sdks-java-io-google-cloud-platform
  • org.apache.beam:beam-sdks-java-io-hadoop-file-system
  • org.apache.beam:beam-sdks-java-io-hadoop-input-format
  • org.apache.beam:beam-sdks-java-io-jdbc

from 2.8.0 to 2.9.0.

I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.

If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.

Have a nice day!

@codecov
Copy link

codecov bot commented Dec 13, 2018

Codecov Report

Merging #1580 into master will decrease coverage by 0.87%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1580      +/-   ##
=========================================
- Coverage   69.88%     69%   -0.88%     
=========================================
  Files         171     171              
  Lines        5246    5246              
  Branches      367     315      -52     
=========================================
- Hits         3666    3620      -46     
- Misses       1580    1626      +46
Impacted Files Coverage Δ
...scala/com/spotify/scio/bigquery/client/Cache.scala 0% <0%> (-58.83%) ⬇️
.../spotify/scio/bigquery/client/BigQueryConfig.scala 0% <0%> (-50%) ⬇️
.../spotify/scio/bigquery/BigQueryPartitionUtil.scala 0% <0%> (-46.16%) ⬇️
...com/spotify/scio/bigquery/types/BigQueryType.scala 37.5% <0%> (-25%) ⬇️
...com/spotify/scio/bigquery/types/TypeProvider.scala 76.59% <0%> (-4.26%) ⬇️
...n/scala/com/spotify/scio/bigquery/BigQueryIO.scala 25% <0%> (-3.85%) ⬇️
...la/com/spotify/scio/bigquery/client/BigQuery.scala 27.02% <0%> (-2.71%) ⬇️
...la/com/spotify/scio/bigquery/client/QueryOps.scala 0.79% <0%> (-0.8%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57238f6...2894255. Read the comment docs.

@jto jto requested review from clairemcginty and regadas December 14, 2018 10:31
@jto
Copy link
Contributor

jto commented Dec 14, 2018

@regadas
Copy link
Contributor

regadas commented Dec 17, 2018

nice! in 2.9.0 they shaded the GRPC and protobuf deps ... so maybe we can remove the overrides we have?

Copy link
Contributor

@regadas regadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can go ahead and merge this.

@jto jto merged commit c40f1bf into spotify:master Dec 17, 2018
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