search

orientechnologies / OrientDB-NET.binary

C#/.NET binary driver for OrientDB
MIT License
93 stars 59 forks source link

The Exception is not explicitly clear enough for debugging #136

Open HuangKBAaron opened 7 years ago

HuangKBAaron commented 7 years ago

It was really took me a long time to feature out lib doesn't support System.UInt32, because of line 164 of \src\OrientDB-Net.binary.Innov8tive\Protocol\Serializers\RecordCSVSerializer.cs throw new NotImplementedException(); It might be better to indicate the type inforamtion, like throw new NotImplementedException(String.Format("type '{1}' of value '{0}' doesn't supported.", value, value.GetType()));

realityenigma commented 7 years ago

This seems like a reasonable request. I will look into implementing it with the new binary driver.