-
Notifications
You must be signed in to change notification settings - Fork 40
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 the conn_id missing in output table #1181
Conversation
Codecov ReportBase: 94.43% // Head: 94.20% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1181 +/- ##
==========================================
- Coverage 94.43% 94.20% -0.23%
==========================================
Files 16 68 +52
Lines 521 3140 +2619
Branches 51 358 +307
==========================================
+ Hits 492 2958 +2466
- Misses 20 120 +100
- Partials 9 62 +53
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
33901fe
to
6f543e8
Compare
6f543e8
to
dab2786
Compare
8f65aaf
to
ce333ae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
7d800eb
to
cd9e293
Compare
b8f16af
to
aff7612
Compare
# Description ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> `conn_id` is missing in output table for transform operator. It should be checked for that <!-- Issues are required for both bug fixes and features. Reference it using one of the following: closes: #ISSUE related: #ISSUE --> closes: #1152 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> - Check if `conn_id` is empty - Push row count of output table to xcom - Fixed some flaky test ## Does this introduce a breaking change? No ### Checklist - [x] Created tests which fail without the change (if possible) - [x] Extended the README / documentation, if necessary Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: rajaths010494 <rajath.srinivasaiah@astronomer.io> Co-authored-by: Pankaj Singh <98807258+pankajastro@users.noreply.github.com> Co-authored-by: Pankaj <pankaj.singh@astronomer.io>
Description
What is the current behavior?
conn_id
is missing in output table for transform operator. It should be checked for thatcloses: #1152
What is the new behavior?
conn_id
is emptyDoes this introduce a breaking change?
No
Checklist