Skip to content
This repository has been 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
wants to merge 1 commit into
base: master
Choose a base branch
from

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