-
Notifications
You must be signed in to change notification settings - Fork 16
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
Adding RTPA and MPO to external_airtable_california_transit_organizations.yml
#3755
Conversation
Please link to the issue this is trying to solve, even if it doesn't full resolve the issue. |
@fsalemi and @vevetron, I believe this will not work. When we download from the Airtable API, Linked Record types will be an array of strings wherein each value is the record ID of another record that is being referred to. Also, I renamed the MPO/RTPA column to RTPA, so I don't believe we will need that former column. Also please work with @csuyat-dot to see if he needs the historic RTPAs and MPOs to be present for dim_organizations prior to this date. That may be a more difficult lift. |
Checking the jsonl of organizations Full Json: |
Ugh I think this broke prod.
Going to revert. @evansiroky I still see both MPO and RTPA in today's airtable organizations table, as in the above. Did we want to delete one of these in airtable or do we want both? Both values could are arrays, either 0 or 1 or multiple values. We should be able to handle it similar to parent_organizations, which is also an array. Though we might need to make it REPEATED and not NULLABLE. This record has multiple parent_organizations:
Line 8 in 3e77334
|
This reverts commit 3e77334, reversing changes made to c1faf49. Co-authored-by: V <[email protected]>
Somebody ran create_external_table at 3:22 pm, good call. I assume it's either @fsalemi or @evansiroky select * from cal-itp-data-infra.staging.stg_transit_database__organizations limit 1 works now. |
@vevetron, We need to have both MPO and RPTA in the table. I should have deleted/renamed the old mpo_rpta in the yml file. I will correct these on Monday. |
Description
Adding RTPA and MPO to external_airtable_california_transit_organizations.yml in order to eventually add these fields to the
dim_organizations
in the GBC warehouse.Resolves #[3751]
Type of change
Post-merge follow-ups
Once this step is completed, Staging, Intermediate and Marts models will be updated to include these fields in the
dim_organizations
.