gnodet commented on PR #1388:
URL: https://github.com/apache/maven/pull/1388#issuecomment-1906038901

   > Yes, that is exactly what is happening, the incremental build basically 
taks "bits" of model and would like to serialize those with DataOutputStream...
   
   I'm not sure we should support this use case, as I think the whole model 
will be serialised anyway.  The lambdas in the list/properties objects do 
retain a reference to the parent up to the Model.  This is needed in order to 
implement the modification methods while keeping the original reference.
   
   So while this may actually make things work, I'm not completely sure we 
should do it...  It may be easier to simplify check if there's a 
`getDelegate()` method, and if so, serialize that object instead.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to