-
Notifications
You must be signed in to change notification settings - Fork 14
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
Build and send prod manifest on each PR #65
base: master
Are you sure you want to change the base?
Conversation
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.
Diffs of tables modified in this pull request: ⚠️ - 3
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 0 (out of 13 compared) | ✔️ | |
💬 Total differing rows | 0 / 783 (0%) | ✔️ | |
💬 Total differing values | 0% | ✔️ | |
View diff details → |
INTEGRATION.BEERS.ORDER_LINES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 900 | 900 (0%) | |
Schema diff | |||
💬 Total columns | 7 | 7 | |
💬 Mismatched columns | 0 | 0 (0 reorders, 0 type changes) | ✔️ |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 900 | 900 (0%) | |
💬 Exclusive PKs | 30 (3.3%) | 30 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 2 (out of 5 compared) | |
|
💬 Total differing rows | 870 / 870 (100%) | |
|
💬 Total differing values | 40% | |
|
View diff details → |
INTEGRATION.BEERS.ORDERS | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 300 | 300 (0%) | |
Schema diff | |||
💬 Total columns | 4 | 5 | |
💬 Mismatched columns | 1 | 2 (0 reorders, 1 type changes) | |
Primary keys diff (order_no) | |||
💬 Distinct PKs | 300 | 300 (0%) | |
💬 Exclusive PKs | 10 (3.3%) | 10 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 2 compared) | |
|
💬 Total differing rows | 290 / 290 (100%) | |
|
💬 Total differing values | 50% | |
|
View diff details → |
Diffs of downstream tables: ⚠️ - 1
INTEGRATION.BEERS.PROMO_DELIVERIES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 180 | 180 (0%) | |
Schema diff | |||
💬 Total columns | 3 | 3 | |
💬 Mismatched columns | 0 | 0 (0 reorders, 0 type changes) | ✔️ |
Primary keys diff (customer_id) | |||
💬 Distinct PKs | 180 | 180 (0%) | |
💬 Exclusive PKs | 180 (100%) | 180 (100%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 0 (out of 2 compared) | ✔️ | |
💬 Total differing rows | 0 / 0 (-/-) | ✔️ | |
💬 Total differing values | -/- | ✔️ | |
View diff details → |
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.
Diffs of tables modified in this pull request: ⚠️ - 3
INTEGRATION.BEERS.ORDERS | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 300 | 300 (0%) | |
Schema diff | |||
💬 Total columns | 4 | 5 | |
💬 Mismatched columns | 1 | 2 (0 reorders, 1 type changes) | |
Primary keys diff (order_no) | |||
💬 Distinct PKs | 300 | 300 (0%) | |
💬 Exclusive PKs | 10 (3.3%) | 10 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 2 compared) | |
|
💬 Total differing rows | 290 / 290 (100%) | |
|
💬 Total differing values | 50% | |
|
View diff details → |
INTEGRATION.BEERS.ORDER_LINES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 900 | 900 (0%) | |
Schema diff | |||
💬 Total columns | 7 | 7 | |
💬 Mismatched columns | 0 | 0 (0 reorders, 0 type changes) | ✔️ |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 900 | 900 (0%) | |
💬 Exclusive PKs | 30 (3.3%) | 30 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 2 (out of 5 compared) | |
|
💬 Total differing rows | 870 / 870 (100%) | |
|
💬 Total differing values | 40% | |
|
View diff details → |
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 0 (out of 13 compared) | ✔️ | |
💬 Total differing rows | 0 / 783 (0%) | ✔️ | |
💬 Total differing values | 0% | ✔️ | |
View diff details → |
Diffs of downstream tables: ⚠️ - 1
INTEGRATION.BEERS.PROMO_DELIVERIES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 180 | 180 (0%) | |
Schema diff | |||
💬 Total columns | 3 | 3 | |
💬 Mismatched columns | 0 | 0 (0 reorders, 0 type changes) | ✔️ |
Primary keys diff (customer_id) | |||
💬 Distinct PKs | 180 | 180 (0%) | |
💬 Exclusive PKs | 180 (100%) | 180 (100%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 0 (out of 2 compared) | ✔️ | |
💬 Total differing rows | 0 / 0 (-/-) | ✔️ | |
💬 Total differing values | -/- | ✔️ | |
View diff details → |
8ee697d
to
1e2cda4
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
f65d460
to
88bf0f1
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
88bf0f1
to
e774f63
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
It guarantees that for each PR we'll always have corresponding prod manifest. That's especially important if prod build runs on long cadence.
e774f63
to
20d5c5e
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
20d5c5e
to
cc72114
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
cc72114
to
d12ec5c
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
<details>
<summary>
<b>Mode downstream: :heavy_check_mark: - 0</b>
</summary>
<br/>
<ul>
</ul>
</details>
d12ec5c
to
3baa528
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
Mode downstream: ✔️ - 0
3baa528
to
2b63759
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
Mode downstream: ✔️ - 0
2b63759
to
747eca9
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
Mode downstream: ✔️ - 0
5924288
to
616eb1f
Compare
50250fa
to
ed2f2bd
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
Mode downstream: ✔️ - 0
ed2f2bd
to
da6e666
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.
Diffs of tables modified in this pull request: ⚠️ - 1
INTEGRATION.BEERS.SALES | |||
---|---|---|---|
main | alex-always-build-prod-manifest | ||
💬 Total rows | 810 | 810 (0%) | |
Schema diff | |||
💬 Total columns | 17 | 18 | |
💬 Mismatched columns | 2 | 3 (0 reorders, 2 type changes) | |
Primary keys diff (order_no, order_line) | |||
💬 Distinct PKs | 810 | 810 (0%) | |
💬 Exclusive PKs | 27 (3.3%) | 27 (3.3%) | |
💬 Null PKs | 0 (0%) | 0 (0%) | ✔️ |
💬 Duplicate PKs | 0 (0%) | 0 (0%) | ✔️ |
Values diff | |||
💬 Differing columns | 1 (out of 13 compared) | |
|
💬 Total differing rows | 783 / 783 (100%) | |
|
💬 Total differing values | 7.7% | |
|
View diff details → |
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.
It guarantees that for each PR we'll always have corresponding prod
manifest.
That's especially important if prod build runs on long cadence.