[ 
https://issues.apache.org/jira/browse/TINKERPOP-1831?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1831.
---------------------------------------
    Resolution: Done

Ended up with a pretty straightforward fix via CTR for this:

https://github.com/apache/tinkerpop/commit/fc866751e11768666ed347f772d86d888a2dec16


> Refactor EventStrategy 
> -----------------------
>
>                 Key: TINKERPOP-1831
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1831
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: structure
>    Affects Versions: 3.2.6
>            Reporter: stephen mallette
>            Assignee: stephen mallette
>            Priority: Minor
>              Labels: breaking
>             Fix For: 3.4.0
>
>
> {{EventStrategy}} has a few issues that could be smoothed out, but not 
> without an allowance for breaking change in the API:
> * For the creation of new properties, an empty detached property is created 
> to represent it - now that detachment is configurable, that doesn't always 
> make sense. For example, if you configured for reference detachment then you 
> would probably want a {{ReferenceProperty}} instead. Not sure how this should 
> be resolved, but it probably needs a change to the eventing API itself
> * Detachment is configured a bit strangely with the use of {{null}} and 
> passing classes for the appropriate detachment factories....would be nicer to 
> have an interface to represent this stuff.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to