fix: CircleCI column naming inconsistency (#8497)#8701
fix: CircleCI column naming inconsistency (#8497)#8701varsis wants to merge 2 commits intoapache:mainfrom
Conversation
CircleCi API returns created_at not created_date.
|
This pr requires some small tweaks to the tests which I thought ran locally but did not. I'll try and correct them tomorrow |
|
|
@klesh The other raw tables also use these values. Would the best approach be to leave those columns as is and pull the |
What other raw tables are you referring to? |
Fixes #8497
Summary
Fixed CircleCI plugin incremental data collection failure due to column naming inconsistency.
Root Cause
created_at/updated_at(snake_case)CreatedDate/UpdatedDate→ DB columnscreated_date/updated_dateCreatedAt/UpdatedAt→ DB columnscreated_at/updated_atcreated_date(didn't exist in those tables)Changes
created_date→created_atCreatedDate/UpdatedDate→CreatedAt/UpdatedAtBreaking Changes
Test Plan