Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Update beginner.md #1825

Merged
merged 1 commit into from Aug 10, 2020
Merged

Update beginner.md #1825

merged 1 commit into from Aug 10, 2020

Conversation

zhouwentong1993
Copy link
Contributor

Create a task needs an owerEmail, Otherwise HTTP Response 400

Create a task needs an owerEmail, Otherwise HTTP Response 400
@zhouwentong1993
Copy link
Contributor Author

Creating Workflow Definition's HTTP request also need a ownerEmail node

@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

Merging #1825 into dev will decrease coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #1825      +/-   ##
============================================
- Coverage     64.80%   64.62%   -0.19%     
+ Complexity     3826     3820       -6     
============================================
  Files           295      295              
  Lines         18180    18180              
  Branches       1653     1653              
============================================
- Hits          11782    11748      -34     
- Misses         5574     5607      +33     
- Partials        824      825       +1     
Impacted Files Coverage Δ Complexity Δ
...ductor/dao/cassandra/CassandraEventHandlerDAO.java 65.78% <0.00%> (-10.53%) 15.00% <0.00%> (ø%)
.../conductor/dao/cassandra/CassandraMetadataDAO.java 53.88% <0.00%> (-5.70%) 24.00% <0.00%> (-1.00%)
...ix/conductor/dao/postgres/PostgresMetadataDAO.java 76.11% <0.00%> (-5.56%) 63.00% <0.00%> (-3.00%)
...tflix/conductor/dao/postgres/PostgresQueueDAO.java 87.60% <0.00%> (-3.31%) 51.00% <0.00%> (-1.00%)
...com/netflix/conductor/service/WorkflowMonitor.java 66.66% <0.00%> (-2.23%) 5.00% <0.00%> (-1.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 7d1caa2...d4d13e8. Read the comment docs.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 4426

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 33 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.2%) to 69.191%

Files with Coverage Reduction New Missed Lines %
postgres-persistence/src/main/java/com/netflix/conductor/dao/postgres/PostgresQueueDAO.java 4 92.56%
cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraEventHandlerDAO.java 8 68.42%
cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraMetadataDAO.java 10 55.44%
postgres-persistence/src/main/java/com/netflix/conductor/dao/postgres/PostgresMetadataDAO.java 11 80.0%
Totals Coverage Status
Change from base Build 4417: -0.2%
Covered Lines: 12579
Relevant Lines: 18180

💛 - Coveralls

@kishorebanala kishorebanala merged commit 6bd8a37 into Netflix:dev Aug 10, 2020
TwoUnderscorez pushed a commit to TwoUnderscorez/conductor that referenced this pull request Jul 23, 2021
Create a task needs an owerEmail, Otherwise HTTP Response 400
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.

None yet

3 participants