Fix unable download twitter dependencies

---

Fixes apache#1962

*Motivation*

There are some twitter dependencies can't download from repo

*Modifications*

- remove some using in StatsLoggerBenchmark.java
- update some config in pom.xml

Descriptions of the changes in this PR:

You can add -Dtwitter to choose add twitter dependencies.

### Motivation

(Explain: why you're making that change, what is the problem you're trying to solve)

### Changes

(Describe: what changes you have made)

Master Issue: apache#1962




Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <zhaijia@apache.org>, Sijie Guo <sijie@apache.org>

This closes apache#1966 from zymap/master

Author: zymap

bookkeeper-server/pom.xml

@@ -241,26 +241,6 @@
     </plugins>
   </build>
   <profiles>
-    <profile>
-      <id>twitter-science-provider</id>
-      <dependencies>
-        <dependency>
-          <groupId>org.apache.bookkeeper.stats</groupId>
-          <artifactId>twitter-science-provider</artifactId>
-          <version>${project.parent.version}</version>
-        </dependency>
-      </dependencies>
-    </profile>
-    <profile>
-      <id>codahale-metrics-provider</id>
-      <dependencies>
-        <dependency>
-          <groupId>org.apache.bookkeeper.stats</groupId>
-          <artifactId>codahale-metrics-provider</artifactId>
-          <version>${project.parent.version}</version>
-        </dependency>
-      </dependencies>
-    </profile>
     <profile>
       <id>twitter-http-server</id>
       <dependencies>

bookkeeper-stats-providers/pom.xml

@@ -26,10 +26,25 @@
   <packaging>pom</packaging>
   <name>Apache BookKeeper :: Stats Providers</name>
   <modules>
-    <module>twitter-finagle-provider</module>
-    <module>twitter-science-provider</module>
-    <module>twitter-ostrich-provider</module>
     <module>codahale-metrics-provider</module>
     <module>prometheus-metrics-provider</module>
   </modules>
+
+  <profiles>
+    <profile>
+      <id>twitter</id>
+      <activation>
+        <property>
+          <name>twitter</name>
+        </property>
+      </activation>
+      <modules>
+        <!-- only building twitter stats providers when `-Dtwitter` is provided -->
+        <module>twitter-finagle-provider</module>
+        <module>twitter-science-provider</module>
+        <module>twitter-ostrich-provider</module>
+      </modules>
+    </profile>
+  </profiles>
+
 </project>

microbenchmarks/pom.xml

@@ -63,16 +63,6 @@
       <artifactId>codahale-metrics-provider</artifactId>
       <version>${project.version}</version>
     </dependency>
-    <dependency>
-      <groupId>org.apache.bookkeeper.stats</groupId>
-      <artifactId>twitter-science-provider</artifactId>
-      <version>${project.version}</version>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.bookkeeper.stats</groupId>
-      <artifactId>twitter-ostrich-provider</artifactId>
-      <version>${project.version}</version>
-    </dependency>
   </dependencies>
   <build>
     <plugins>

microbenchmarks/src/main/java/org/apache/bookkeeper/stats/StatsLoggerBenchmark.java

@@ -30,8 +30,6 @@
 import org.apache.bookkeeper.stats.codahale.CodahaleMetricsProvider;
 import org.apache.bookkeeper.stats.codahale.FastCodahaleMetricsProvider;
 import org.apache.bookkeeper.stats.prometheus.PrometheusMetricsProvider;
-import org.apache.bookkeeper.stats.twitter.ostrich.OstrichProvider;
-import org.apache.bookkeeper.stats.twitter.science.TwitterStatsProvider;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.Fork;
 import org.openjdk.jmh.annotations.Level;
@@ -60,8 +58,6 @@ public class StatsLoggerBenchmark {
         providers.put("Prometheus", PrometheusMetricsProvider::new);
         providers.put("Codahale", CodahaleMetricsProvider::new);
         providers.put("FastCodahale", FastCodahaleMetricsProvider::new);
-        providers.put("Twitter", TwitterStatsProvider::new);
-        providers.put("Ostrich", OstrichProvider::new);
     }
 
     @State(Scope.Benchmark)