Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added flag prohibiting GraphDiff from ever removing collection elements ... #107

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Oct 28, 2014

...from the database (owned or associated). An attempt to provide a config for issue #105. Currently this still loads (has to) all elements of the collection from the database (to be able to compare, what's new/updated etc., while attaching). So this implementation quite definitely has performance issues for the case it is intended to support, because the number of elements in the collection would only ever increase. Probably an (unchecked) append-only mode for collections would be of more value to support this case.

@refactorthis: Do you think this is a useful feature, so I'd be worth investing some time exploring an append-only mode? Would you prefer to do this via the fluent API and/or attribute mappings instead of a global configuration to be more flexible / useful?

…ts from the database (owned or associated)
@odimperio
Copy link

Hey Andy , i need insert child objects and update it if is need and delete only the relation between the "father" and the "child" but not delete the child object.
This feature sounds cool, you think this solves my problem?
When i use AssociatedCollection the params of the childrens are empty or null.
I wait for your answer.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant