-
Notifications
You must be signed in to change notification settings - Fork 3.6k
fix: correct default value backfill during AddField #45644
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
Conversation
Signed-off-by: zhenshan.cao <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: czs007 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
[ci-v2-notice]
To rerun ci-v2 checks, comment with:
If you have any questions or requests, please contact @zhikunyao. |
|
[INFO] PR Label Summary by Default
[WARNING] Milestone not set
You can set milestone by commenting: Use /refresh-label to update related check and label manually |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 2.6 #45644 +/- ##
==========================================
- Coverage 76.99% 76.62% -0.38%
==========================================
Files 1700 1856 +156
Lines 262533 287687 +25154
==========================================
+ Hits 202142 220438 +18296
- Misses 53550 59911 +6361
- Partials 6841 7338 +497
🚀 New features to boost your workflow:
|
|
/lgtm |
|
[INFO] PR Label Summary by Default
Use /refresh-label to update related check and label manually |
issue: #44585
pr: #45634