Join Slack
Powered by
<@U06CE5VC812> - Wanted to call out we've merged y...
# contributing
a
aaronsteers
10/01/2021, 7:19 PM
@artem_vysotsky
- Wanted to call out we've merged your MR after very in-depth discussion of all the pros and cons. I appreciate your patience and understanding as we wrestled with this - a messy topic and I think there are valid points on all sides of the debate. 馃У
Don't fail when schema_at_breadcrumb does not have "properties" (!120) 路 Merge requests 路 Meltano / Meltano SDK for Singer Taps and Targets 路 GitLab
aaronsteers
10/01/2021, 7:20 PM
Since this is a niche case, I don't have a high confidence that all selection scenarios will work as-expected, but we can commit to working with you to support these issues if/as they come up.
aaronsteers
10/01/2021, 7:27 PM
We do think targets should handle these gracefully and I've logged a discussion item for our new Singer Spec Working Group:
Best Practice Target Capability: Data Type Failsafe
aaronsteers
10/01/2021, 7:28 PM
cc
@edgar_ramirez_mondragon
and
@taylor
Open in Slack
Previous
Next