search

Netflix / genie

Distributed Big Data Orchestration Service
https://netflix.github.io/genie
Apache License 2.0
1.71k stars 367 forks source link

Add missing fields to Titus request for job grouping #1076

Closed tgianos closed 3 years ago

tgianos commented 3 years ago

stack, detail, sequence

Also add exception to adapter API

coveralls commented 3 years ago

Coverage Status

Coverage decreased (-0.03%) to 93.901% when pulling 4317b8d289330d29f55d458289bfb1a44d8327f3 on tgianos:GENIE-712 into a6d510e822e629fc79efd534369d251ab0b55484 on Netflix:master.

codecov[bot] commented 3 years ago

Codecov Report

Merging #1076 (4317b8d) into master (a6d510e) will decrease coverage by 0.02%. The diff coverage is 75.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1076      +/-   ##
============================================
- Coverage     90.90%   90.87%   -0.03%     
- Complexity     3638     3641       +3     
============================================
  Files           452      452              
  Lines         14059    14073      +14     
  Branches        983      984       +1     
============================================
+ Hits          12780    12789       +9     
- Misses          845      850       +5     
  Partials        434      434
Impacted Files Coverage Δ Complexity Δ
...web/agent/launchers/dtos/TitusBatchJobRequest.java 59.32% <16.66%> (-4.83%) 8.00 <0.00> (ø)
...b/agent/launchers/impl/TitusAgentLauncherImpl.java 93.58% <100.00%> (+0.17%) 10.00 <1.00> (ø)
...e/web/properties/TitusAgentLauncherProperties.java 100.00% <100.00%> (ø) 35.00 <3.00> (+3.00)

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 a6d510e...4317b8d. Read the comment docs.