Arthur Vickers

Results 1128 comments of Arthur Vickers

Note: see also #10465, which is very similar but comes from a slightly different angle.

@roji Thanks for checking this. It's unfortunate, but I think it re-enforces that we need to retain both collection-include ehaviors for now.

Note for triage: passes with ICollection nav; fails with IReadOnlyCollection nav, with 6.0 on in-memory and SQL Server. ```C# public class RootEntity { [DatabaseGenerated(DatabaseGeneratedOption.None)] public int Id { get; set;...

Note for triage: not a regression from 5.0. Copy/paste repro: ```C# public class RootEntity { public int Id { get; set; } public IReadOnlyCollection ChildEntites { get; set; } }...

Note from triage: consider improving functionality of collection accessor.

@midczak Try converting directly to binary. This will avoid EF Core composing two converters together to go from your type to Guid, and then Guid to binary.

@AndriySvyryd Looks like this could be an issue with the update pipeline collapsing Added/Deleted owned entities into a single update when all values of the Added entity are CLR defaults...

Note from triage: make sure that truncation does not occur in cases where inferred type mapping has a max length that may not accommodate values from both sides.