Fix possible NullPointerException in Connection.shardId()

Bouncheck · avelanarius · commit b3f3ebaf161b · 2023-02-20T13:12:58.000+01:00
There is no need to call getShardingInfo() because when shardId is set to null getShardingInfo() is also null. Fixes #180.
diff --git a/driver-core/src/main/java/com/datastax/driver/core/Connection.java b/driver-core/src/main/java/com/datastax/driver/core/Connection.java
@@ -1047,7 +1047,7 @@ boolean setOwner(Owner owner) {
   }
 
   public int shardId() {
-    return shardId == null || getHost().getShardingInfo() == null ? 0 : shardId;
+    return shardId == null ? 0 : shardId;
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -1047,7 +1047,7 @@ boolean setOwner(Owner owner) {`
`1047`	`1047`	`}`
`1048`	`1048`
`1049`	`1049`	`public int shardId() {`
`1050`		`- return shardId == null \|\| getHost().getShardingInfo() == null ? 0 : shardId;`
	`1050`	`+ return shardId == null ? 0 : shardId;`
`1051`	`1051`	`}`
`1052`	`1052`
`1053`	`1053`	`/**`