Update default read source setting (#5)

Author: linxGnu

mssqlx.go

@@ -1432,7 +1432,7 @@ func ConnectMasterSlaves(driverName string, masterDSNs []string, slaveDSNs []str
 	// default cluster options
 	opts := &clusterOptions{
 		isWsrep:         false,
-		readQuerySource: ReadQuerySourceSlaves,
+		readQuerySource: ReadQuerySourceAll,
 	}
 
 	for _, optFn := range options {

mssqlx_test.go

@@ -467,12 +467,12 @@ func TestConnectMasterSlave(t *testing.T) {
 
 		// check read-query source
 		db, _ = ConnectMasterSlaves(driver, nil, nil)
-		if db.readBalancer() != db.slaves {
+		if db.readBalancer() != db.all {
 			t.Fatal("Initialize master slave fail")
 		}
 
-		db, _ = ConnectMasterSlaves(driver, nil, nil, WithReadQuerySource(ReadQuerySourceAll))
-		if db.readBalancer() != db.all {
+		db, _ = ConnectMasterSlaves(driver, nil, nil, WithReadQuerySource(ReadQuerySourceSlaves))
+		if db.readBalancer() != db.slaves {
 			t.Fatal("Initialize master slave fail")
 		}
 	}
@@ -1817,6 +1817,8 @@ func TestStressQueries(t *testing.T) {
 	}
 
 	_RunWithSchema(schema, t, func(db *DBs, t *testing.T) {
+		require.Equal(t, ReadQuerySourceAll, db.readQuerySource)
+
 		ch := make(chan struct{}, 100)
 
 		type StressType struct {

options.go

@@ -4,13 +4,15 @@ package mssqlx
 type ReadQuerySource int
 
 const (
-	// ReadQuerySourceSlaves indicates: read-queries will be distributed only among slaves.
-	// This is default value.
+	// ReadQuerySourceSlaves setting indicates: read-queries will be distributed only among slaves.
 	//
 	// Note: there is no option for Master. One could use functions like `QueryMaster`, etc
 	// to query from masters only.
 	ReadQuerySourceSlaves ReadQuerySource = iota
-	// ReadQuerySourceAll indicates: read-queries will be distributed among both masters and slaves.
+
+	// ReadQuerySourceAll setting indicates: read-queries will be distributed among both masters and slaves.
+	//
+	// Note: this is default setting.
 	ReadQuerySourceAll
 )