Skip to content
This repository was archived by the owner on Dec 6, 2019. It is now read-only.

Bug 1574490 - Add support for reading from BigQuery views to EventsTo…#255

Draft
akkomar wants to merge 1 commit intomasterfrom
amplitude_gcp
Draft

Bug 1574490 - Add support for reading from BigQuery views to EventsTo…#255
akkomar wants to merge 1 commit intomasterfrom
amplitude_gcp

Conversation

@akkomar
Copy link
Contributor

@akkomar akkomar commented Dec 2, 2019

…Amplitude job

This adds support to read from views introduced in mozilla/bigquery-etl#510 and can be used as an alternative solution if S3 ingestion takes too long to set up.

…Amplitude job

This adds support to read from views introduced in mozilla/bigquery-etl#510.
This can be used as an alternative solution if S3 ingestion  takes too long to set up.
@codecov-io
Copy link

Codecov Report

Merging #255 into master will decrease coverage by 0.37%.
The diff coverage is 22.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #255      +/-   ##
==========================================
- Coverage   69.34%   68.96%   -0.38%     
==========================================
  Files          34       34              
  Lines        1543     1563      +20     
  Branches       83       86       +3     
==========================================
+ Hits         1070     1078       +8     
- Misses        473      485      +12
Impacted Files Coverage Δ
...ozilla/telemetry/streaming/EventsToAmplitude.scala 62.31% <22.22%> (-3.79%) ⬇️

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 b92f766...0c17971. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants