[Discussion/Feature request] Adding `database` field to Linking Iterables by field on import

[marc-vdm]

[Discussion/Feature request] Adding `database` field to Linking Iterables by field on import

[cmutel]

I didn't understand this at first, as one always needs to specify the database against which one is linking. However, I think the problem that @marc-vdm identified is that your source data could include links to multiple secondary databases, and he wants to be able to avoid incorrect matches.

I am reluctant to add this to the default implementation - it will break more than it fixes in most use cases. But I agree that we should make it more clear how to make changes to the fields passed in as a default, both in the documentation and source code. We could also add a convenience function to make this simpler (now you would have to find the curried strategy by index if using the default strategy list).

[marc-vdm]

For clarity:

problem that @marc-vdm identified is that your source data could include links to multiple secondary databases, and he wants to be able to avoid incorrect matches.

Yes, that's indeed what I meant.

However, we continued some discussion internally over here in parallel, and I agree, this should probably not become a default field in BW.

I'll leave this open if you want to keep this for tracking the documentation improvement, otherwise, just close it.