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

feat: add Retry-After header as FlowFile attribute#67

Merged
bednar merged 5 commits intomasterfrom
retry_after_header
Mar 29, 2022
Merged

feat: add Retry-After header as FlowFile attribute#67
bednar merged 5 commits intomasterfrom
retry_after_header

Conversation

@bednar
Copy link
Copy Markdown
Contributor

@bednar bednar commented Mar 23, 2022

The processors put the Retry-After header value as a FlowFile Attribute (influxdb.retry-after) before sending the FlowFile to the “Retry” output.

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Sign CLA (if not already signed)

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 23, 2022

Codecov Report

Merging #67 (69bca78) into master (6cd9b34) will decrease coverage by 0.04%.
The diff coverage is 86.66%.

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
- Coverage     93.84%   93.79%   -0.05%     
  Complexity      131      131              
============================================
  Files            27       27              
  Lines          1966     1983      +17     
  Branches        137      140       +3     
============================================
+ Hits           1845     1860      +15     
- Misses           67       69       +2     
  Partials         54       54              
Impacted Files Coverage Δ
...sors/internal/AbstractInfluxDatabaseProcessor.java 95.00% <ø> (ø)
...rs/internal/AbstractInfluxDatabaseProcessor_2.java 86.73% <77.77%> (-0.91%) ⬇️
...ata/nifi/processors/PutInfluxDatabaseRecord_2.java 100.00% <100.00%> (ø)
...nfluxdata/nifi/processors/PutInfluxDatabase_2.java 95.65% <100.00%> (+0.26%) ⬆️

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 6cd9b34...69bca78. Read the comment docs.

@bednar bednar requested a review from rhajek March 23, 2022 11:56
@bednar bednar marked this pull request as ready for review March 23, 2022 11:56
@bednar bednar merged commit 85fce89 into master Mar 29, 2022
@bednar bednar deleted the retry_after_header branch March 29, 2022 12:04
@bednar bednar added this to the 1.16.0 milestone Mar 29, 2022
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.

3 participants