Add more comments on linked attribute limitations

[Tim Beale]

RB+ with the suggested changes.diff attached. I attached an updated
version of the original patch too.

Suggested changes:
- replmd_delete_internals(): I just moved the comment up a bit. (I found
it a bit confusing to talk about not finding backlinks when we clearly
had found one).
- replmd_process_linked_attribute(): this function is complex enough,
and replmd_delete_link_value() should already cover the DRS code-path.
It might be better sticking a comment in linked_attributes.c instead.
- minor elaboration on "filtering at runtime" and fixed typo.

On 30/10/18 4:01 PM, Andrew Bartlett wrote:
> G'Day,
>
> While reviewing https://gitlab.com/samba-team/samba/merge_requests/90 I
> forgot how bad the limitations on our linked attribute handling are.
>
> This commit just tries to add some more warnings that might help remind
> the next person trying to remember why we can't have nice things.
>
> Please review so I can push with metze's patches.
>
> Thanks!
>
> Andrew Bartlett
-------------- next part --------------
A non-text attachment was scrubbed...
Name: changes.diff
Type: text/x-patch
Size: 4920 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20181030/97bf107f/changes.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dsdb-Add-comments-explaining-the-limitations-of-our-.patch
Type: text/x-patch
Size: 5218 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20181030/97bf107f/0001-dsdb-Add-comments-explaining-the-limitations-of-our-.bin>