Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[plugin-azure-data-factory] Add ability to collect pipeline runs #2198

Merged
merged 1 commit into from
Nov 16, 2021

Conversation

valfirst
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Nov 15, 2021

Codecov Report

Merging #2198 (4978be0) into master (61e46de) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2198   +/-   ##
=========================================
  Coverage     96.22%   96.23%           
- Complexity     5380     5384    +4     
=========================================
  Files           764      764           
  Lines         15394    15426   +32     
  Branches       1025     1025           
=========================================
+ Hits          14813    14845   +32     
  Misses          451      451           
  Partials        130      130           
Impacted Files Coverage Δ
...a/org/vividus/azure/util/InnersJacksonAdapter.java 100.00% <100.00%> (ø)
...idus/azure/datafactory/steps/DataFactorySteps.java 100.00% <100.00%> (ø)
...idus/azure/eventgrid/EventGridManagementSteps.java 100.00% <100.00%> (ø)
...s/azure/storage/StorageAccountManagementSteps.java 100.00% <100.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 61e46de...4978be0. Read the comment docs.

this.filterSetter = filterSetter;
}

public void setFilters(RunFilterParameters filterParameters, OffsetDateTime filterValue)
Copy link
Member

@ikalinin1 ikalinin1 Nov 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setFilters looks confusing, the method is applying a filter to the RunFilterParameters

@valfirst valfirst force-pushed the add-ability-to-collect-pipeline-runs branch from 8329f2d to fd0f11e Compare November 16, 2021 07:04
@valfirst valfirst force-pushed the add-ability-to-collect-pipeline-runs branch from fd0f11e to 4978be0 Compare November 16, 2021 08:02
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@valfirst valfirst merged commit c46b670 into master Nov 16, 2021
@valfirst valfirst deleted the add-ability-to-collect-pipeline-runs branch November 16, 2021 12:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants