Build and send prod manifest on each PR#65
Open
alexandermorozov wants to merge 2 commits intomasterfrom
Open
Conversation
![datafold[bot]](https://avatars.githubusercontent.com/in/103851?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
3 times, most recently
from
8ee697d to
1e2cda4
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
f65d460 to
88bf0f1
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
88bf0f1 to
e774f63
Compare
There was a problem hiding this comment.
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.
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
e774f63 to
20d5c5e
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
20d5c5e to
cc72114
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
cc72114 to
d12ec5c
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
d12ec5c to
3baa528
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
3baa528 to
2b63759
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
2b63759 to
747eca9
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
3 times, most recently
from
5924288 to
616eb1f
Compare
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
2 times, most recently
from
50250fa to
ed2f2bd
Compare
There was a problem hiding this comment.
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 → | |||
alexandermorozov
force-pushed
the
alex-always-build-prod-manifest
branch
from
ed2f2bd to
da6e666
Compare
There was a problem hiding this comment.
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 → | |||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It guarantees that for each PR we'll always have corresponding prod
manifest.
That's especially important if prod build runs on long cadence.