Commented

Author: bahusoid

src/NHibernate/Connection/IConnectionAccess.cs

@@ -6,6 +6,7 @@ namespace NHibernate.Connection
 	/// <summary>
 	/// Provides centralized access to connections.  Centralized to hide the complexity of accounting for contextual
 	/// (multi-tenant) versus non-contextual access.
+	/// Implementation must be serializable
 	/// </summary>
 	public partial interface IConnectionAccess
 	{

src/NHibernate/MultiTenancy/AbstractMultiTenantConnectionProvider.cs

@@ -12,9 +12,15 @@ namespace NHibernate.MultiTenancy
 	[Serializable]
 	public abstract partial class AbstractMultiTenantConnectionProvider : IMultiTenantConnectionProvider
 	{
+		/// <summary>
+		/// Tenant connection string
+		/// </summary>
 		protected abstract string TenantConnectionString { get; }
+
+		/// <inheritdoc />
 		public abstract string TenantIdentifier { get; }
 
+		/// <inheritdoc />
 		public IConnectionAccess GetConnectionAccess()
 		{
 			//TODO 6.0: Remove check

src/NHibernate/MultiTenancy/IMultiTenantConnectionProvider.cs

@@ -8,8 +8,15 @@ namespace NHibernate.MultiTenancy
 	/// </summary>
 	public interface IMultiTenantConnectionProvider
 	{
+		/// <summary>
+		/// Tenant identifier must uniquely identify tenant (is used for data separation in cache)
+		/// Note: Among other things this value is used for data separation between tenants in cache so not unique value will leak data to other tenants 
+		/// </summary>
 		string TenantIdentifier { get; } 
 
+		/// <summary>
+		/// Tenant connection access
+		/// </summary>
 		IConnectionAccess GetConnectionAccess();
 	}
 }

src/NHibernate/MultiTenancy/MultiTenancyStrategy.cs

@@ -1,5 +1,9 @@
 namespace NHibernate.MultiTenancy
 {
+	/// <summary>
+	/// Strategy for multi-tenancy
+	/// <seealso cref="NHibernate.Cfg.Environment.MultiTenant"/>
+	/// </summary>
 	public enum MultiTenancyStrategy
 	{
 		/// <summary>

src/NHibernate/MultiTenancy/TenantConfiguration.cs

@@ -7,9 +7,20 @@ namespace NHibernate.MultiTenancy
 	/// </summary>
 	public class TenantConfiguration
 	{
+		/// <summary>
+		/// Tenant identifier must uniquely identify tenant
+		/// Note: Among other things this value is used for data separation between tenants in cache so not unique value will leak data to other tenants 
+		/// </summary>
 		public string TenantIdentifier { get; set; }
+
+		/// <summary>
+		/// Tenant connection access. Required for <seealso cref="MultiTenancyStrategy.Database"/> multi-tenancy strategy.
+		/// </summary>
 		public IConnectionAccess ConnectionAccess { get; set; }
 
+		/// <summary>
+		/// Creates tenant configuration for <seealso cref="MultiTenancyStrategy.Database"/> multi-tenancy strategy.
+		/// </summary>
 		public TenantConfiguration(IMultiTenantConnectionProvider tenantConnectionProvider)
 		{
 			TenantIdentifier = tenantConnectionProvider.TenantIdentifier;