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

Update V3__queue_add_priority.sql #1616

Closed
wants to merge 1 commit into from
Closed

Update V3__queue_add_priority.sql #1616

wants to merge 1 commit into from

Conversation

andyyumiao
Copy link
Contributor

Remove SQL grammar when adding column : 'IF NOT EXISTS' , it's for PostgreSQL.

@codecov
Copy link

codecov bot commented Mar 29, 2020

Codecov Report

Merging #1616 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #1616     +/-   ##
===========================================
- Coverage     64.16%   64.06%   -0.1%     
  Complexity     3415     3415             
===========================================
  Files           268      268             
  Lines         16154    16154             
  Branches       1446     1446             
===========================================
- Hits          10365    10349     -16     
- Misses         5071     5087     +16     
  Partials        718      718
Impacted Files Coverage Δ Complexity Δ
...ductor/dao/cassandra/CassandraEventHandlerDAO.java 68.49% <0%> (-10.96%) 15% <0%> (ø)
.../conductor/dao/cassandra/CassandraMetadataDAO.java 54.73% <0%> (-4.22%) 24% <0%> (ø)

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 a2c37c5...bf9b6be. Read the comment docs.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3890

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 16 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 68.922%

Files with Coverage Reduction New Missed Lines %
cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraEventHandlerDAO.java 8 69.86%
cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraMetadataDAO.java 8 55.79%
Totals Coverage Status
Change from base Build 3866: -0.1%
Covered Lines: 11348
Relevant Lines: 16465

💛 - Coveralls

@apanicker-nflx apanicker-nflx changed the base branch from master to dev April 3, 2020 01:07
@apanicker-nflx
Copy link
Collaborator

Please resolve conflicts.

@andyyumiao andyyumiao closed this Apr 3, 2020
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