m-lab / etl

M-Lab ingestion pipeline
Apache License 2.0
22 stars 7 forks source link

Add NDT5ResultRowStandardColumns #956

Closed gfr10598 closed 4 years ago

gfr10598 commented 4 years ago

We want to make NDT5 conform to standard columns, but doing so directly would break the legacy parser pipeline.

This adds a V1 version that implements standard columns.

The intent is then to use the original for legacy parsing, and the new one for Gardener 2.0 parsing.


This change is Reviewable

coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 6231


Changes Missing Coverage Covered Lines Changed/Added Lines %
schema/bindata.go 12 14 85.71%
schema/ndt5_result.go 9 11 81.82%
<!-- Total: 21 25 84.0% -->
Totals Coverage Status
Change from base Build 6209: 0.2%
Covered Lines: 3171
Relevant Lines: 4774

💛 - Coveralls