You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Singer's GitHub tap is adding an _sdc_repository field for streams that don't get it from the API as an easy-to-use identifier to streams to their parent repo.
Hi, @ericboucher. I haven't looked in advance at what the overloaded repo and org fields would contain in the other stream. We imagined those repo and org attributes to be generic and pretty much universal, unless there's a reason to do otherwise. (Renaming would cause some headache at this point but I don't think it's fully off the table.)
Personally, I have considered a generic a _sdc_parent_keys column or sdc_partition_context, or similar, but I'd like to stay away from the _sdc_repository name since that is specifically unique to tap-github.
Singer's GitHub tap is adding an
_sdc_repository
field for streams that don't get it from the API as an easy-to-use identifier to streams to their parent repo.See https://github.com/singer-io/tap-github/blob/master/tap_github/__init__.py#L927
Should we do something similar with the Meltano taps?
At the moment,
repo
andorg
keys get added, so we could decide to leverage that instead.One caveat, some streams actually have fields called
repo
andorg
that could be incompatible (see my upcoming Events stream PR).The text was updated successfully, but these errors were encountered: