Don't use public API to access metadata from ArrayBackedResultSet.

olim7t · olim7t · commit e3bfb470485d · 2015-12-01T14:36:27.000+01:00
This avoids triggering Cluster.init(), which acquires a lock.
diff --git a/driver-core/src/main/java/com/datastax/driver/core/ArrayBackedResultSet.java b/driver-core/src/main/java/com/datastax/driver/core/ArrayBackedResultSet.java
@@ -68,7 +68,7 @@ static ArrayBackedResultSet fromMessage(Responses.Result msg, SessionManager ses
                 }
 
                 Token.Factory tokenFactory = (session == null) ? null
-                    : session.getCluster().getMetadata().tokenFactory();
+                    : session.getCluster().manager.metadata.tokenFactory();
 
                 // info can be null only for internal calls, but we don't page those. We assert
                 // this explicitly because MultiPage implementation don't support info == null.

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ static ArrayBackedResultSet fromMessage(Responses.Result msg, SessionManager ses`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`Token.Factory tokenFactory = (session == null) ? null`
`71`		`- : session.getCluster().getMetadata().tokenFactory();`
	`71`	`+ : session.getCluster().manager.metadata.tokenFactory();`
`72`	`72`
`73`	`73`	`// info can be null only for internal calls, but we don't page those. We assert`
`74`	`74`	`// this explicitly because MultiPage implementation don't support info == null.`