Skip to content

[to dev/1.3] fix: prevent NPE when isFinished() is called before DataDriver init#17441

Merged
JackieTien97 merged 3 commits intodev/1.3from
fixNPE-0408-1.3
Apr 8, 2026
Merged

[to dev/1.3] fix: prevent NPE when isFinished() is called before DataDriver init#17441
JackieTien97 merged 3 commits intodev/1.3from
fixNPE-0408-1.3

Conversation

@shuwenwei
Copy link
Copy Markdown
Member

(cherry picked from (#17440))

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 41.51%. Comparing base (28c4e02) to head (1a0fe5a).
⚠️ Report is 1 commits behind head on dev/1.3.

Files with missing lines Patch % Lines
.../db/queryengine/execution/driver/SchemaDriver.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             dev/1.3   #17441   +/-   ##
==========================================
  Coverage      41.51%   41.51%           
  Complexity       227      227           
==========================================
  Files           3596     3596           
  Lines         236857   236859    +2     
  Branches       28677    28677           
==========================================
+ Hits           98332    98333    +1     
- Misses        138525   138526    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JackieTien97 JackieTien97 merged commit 230cd63 into dev/1.3 Apr 8, 2026
19 of 20 checks passed
@JackieTien97 JackieTien97 deleted the fixNPE-0408-1.3 branch April 8, 2026 06:12
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.

2 participants