NH-4017 - Npgsql v3 expects TimeSpan for time.

Author: fredericDelaporte

src/NHibernate/Driver/NpgsqlDriver.cs

@@ -1,5 +1,7 @@
+using System;
 using System.Data;
 using System.Data.Common;
+using System.Linq;
 
 namespace NHibernate.Driver
 {
@@ -84,5 +86,9 @@ protected override void InitializeParameter(DbParameter dbParam, string name, Sq
 			if (sqlType.DbType == DbType.Currency)
 				dbParam.DbType = DbType.Decimal;
 		}
+
+		// Prior to v3, Npgsql was expecting DateTime for time.
+		// https://github.com/npgsql/npgsql/issues/347
+		public override bool RequiresTimeSpanForTime => (DriverVersion?.Major ?? 3) >= 3;
 	}
 }

src/NHibernate/Driver/ReflectionBasedDriver.cs

@@ -1,3 +1,4 @@
+using System;
 using System.Data.Common;
 using NHibernate.Util;
 
@@ -12,6 +13,11 @@ public abstract class ReflectionBasedDriver : DriverBase
 
 		private readonly IDriveConnectionCommandProvider connectionCommandProvider;
 
+		/// <summary>
+		/// If the driver use a third party driver (not a .Net Framework DbProvider), its assembly version.
+		/// </summary>
+		protected Version DriverVersion { get; } 
+
 		/// <summary>
 		/// Initializes a new instance of <see cref="ReflectionBasedDriver" /> with
 		/// type names that are loaded from the specified assembly.
@@ -51,6 +57,7 @@ protected ReflectionBasedDriver(string providerInvariantName, string driverAssem
 			else
 			{
 				connectionCommandProvider = new ReflectionDriveConnectionCommandProvider(connectionType, commandType);
+				DriverVersion = connectionType.Assembly.GetName().Version;
 			}
 		}