Upgrade the bookkeeper storage client dependency to the official book…

…keeper version (#1328)

This removes the temp dependency in `pulsar-functions-instance`

pom.xml

@@ -230,6 +230,12 @@ flexible messaging model and an intuitive client API.</description>
         </exclusions>
       </dependency>
 
+      <dependency>
+        <groupId>org.apache.bookkeeper</groupId>
+        <artifactId>stream-storage-java-client</artifactId>
+        <version>${bookkeeper.version}</version>
+      </dependency>
+
       <dependency>
         <groupId>org.apache.bookkeeper</groupId>
         <artifactId>bookkeeper-bookkeeper-stats-api</artifactId>

pulsar-functions/instance/pom.xml

@@ -61,7 +61,6 @@
     <dependency>
       <groupId>org.apache.bookkeeper</groupId>
       <artifactId>stream-storage-java-client</artifactId>
-      <version>0.6.0</version>
     </dependency>
 
     <dependency>
@@ -86,15 +85,4 @@
 
   </dependencies>
 
-  <!-- current the stream storage artifacts are hosted at streamlio. will give rid of this once it is proposed to merge to bookkeeper -->
-  <repositories>
-    <repository>
-        <id>stream-storage-mvn-repo</id>
-        <url>https://raw.githubusercontent.com/sijie/artifacts/mvn-repo/</url>
-        <snapshots>
-            <enabled>true</enabled>
-        </snapshots>
-    </repository>
-  </repositories>
-
 </project>