fsprojects / FSharp.Azure.Storage

F# API for using Microsoft Azure Table Storage service
MIT License
75 stars 16 forks source link

Make small records of struct type #41

Open xperiandri opened 3 years ago

xperiandri commented 3 years ago

Records EntityIdentifier, OperationResult, EntityMetadata have only 2 fields which means that making them structs is efficient option.

daniel-chambers commented 3 years ago

Pretty sure when I started this library, you couldn't make records value types. I'm not fundamentally opposed to the idea, but it would be a gentle breaking change.

Did you have some specific performance issues with them as reference types?