Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix NullPointerException when retrieving record with default value #533

Merged
merged 6 commits into from
Jun 15, 2022

Conversation

kaneeldias
Copy link
Contributor

Purpose

$subject

Fixes: ballerina-platform/ballerina-library#2985

Examples

Checklist

  • Linked to an issue
  • Updated the specification
  • Updated the changelog
  • Added tests

@codecov
Copy link

codecov bot commented Jun 13, 2022

Codecov Report

Merging #533 (4995eee) into master (d76321f) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #533      +/-   ##
==========================================
+ Coverage   80.25%   80.30%   +0.04%     
==========================================
  Files          38       38              
  Lines        3611     3615       +4     
  Branches      702      703       +1     
==========================================
+ Hits         2898     2903       +5     
+ Misses        490      489       -1     
  Partials      223      223              
Impacted Files Coverage Δ
...c/main/java/io/ballerina/stdlib/sql/Constants.java 100.00% <ø> (ø)
...main/java/io/ballerina/stdlib/sql/utils/Utils.java 76.61% <ø> (ø)
...allerina/stdlib/sql/nativeimpl/QueryProcessor.java 82.35% <100.00%> (+0.14%) ⬆️
...allerina/stdlib/sql/utils/RecordIteratorUtils.java 72.72% <100.00%> (+2.72%) ⬆️
ballerina/types.bal 90.23% <0.00%> (+0.33%) ⬆️

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 d76321f...4995eee. Read the comment docs.

@kaneeldias kaneeldias marked this pull request as ready for review June 13, 2022 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NullPointerException when retrieving record with default value
2 participants