You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by aa...@apache.org on 2021/04/17 07:40:55 UTC

[pulsar] branch master updated: Fix compilation failure in master branch (#10250)

This is an automated email from the ASF dual-hosted git repository.

aahmed pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new f986cfe  Fix compilation failure in master branch (#10250)
f986cfe is described below

commit f986cfe42ef58add1794af9c727795b9e7fcde31
Author: Lari Hotari <lh...@users.noreply.github.com>
AuthorDate: Sat Apr 17 10:40:00 2021 +0300

    Fix compilation failure in master branch (#10250)
    
    - possibly a merge conflict in #10201
---
 .../pulsar/broker/resourcegroup/ResourceGroupServiceTest.java     | 8 --------
 .../broker/resourcegroup/ResourceGroupUsageAggregationTest.java   | 2 --
 2 files changed, 10 deletions(-)

diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupServiceTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupServiceTest.java
index ebbdea3..df2e6a4 100644
--- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupServiceTest.java
+++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupServiceTest.java
@@ -242,16 +242,8 @@ public class ResourceGroupServiceTest extends MockedPulsarServiceBaseTest {
 
     private static final Logger log = LoggerFactory.getLogger(ResourceGroupServiceTest.class);
 
-    private static final String TEST_PRODUCE_CONSUME_TOPIC =
-            "non-persistent://prod-cons-tenant/prod-cons-ns/prod-cons-topic";
-
-    // Set up configuration that is required to be able to use the transport manager.
-    // private static final String INTERNAL_USAGE_TOPIC = "non-persistent://rusage-tenant/rusage-ns/rusage-topic";
-    private static final String INTERNAL_USAGE_TOPIC = TEST_PRODUCE_CONSUME_TOPIC;
-
     private static final int PUBLISH_INTERVAL_SECS = 500;
     private void prepareData() throws PulsarAdminException {
-        this.conf.setResourceUsageTransportPublishTopicName(INTERNAL_USAGE_TOPIC);
         this.conf.setResourceUsageTransportPublishIntervalInSecs(PUBLISH_INTERVAL_SECS);
         admin.clusters().createCluster("test", new ClusterData(pulsar.getBrokerServiceUrl()));
     }
diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupUsageAggregationTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupUsageAggregationTest.java
index 76caf4b..a8892b2 100644
--- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupUsageAggregationTest.java
+++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupUsageAggregationTest.java
@@ -239,7 +239,6 @@ public class ResourceGroupUsageAggregationTest extends ProducerConsumerBase {
     final String NsName = "test";
     final String TenantAndNsName = TenantName + "/" + NsName;
     final String TestProduceConsumeTopicName = "/test/prod-cons-topic";
-    final String INTERNAL_TOPIC = "non-persistent://" + TenantAndNsName + "/resource-usage-internal";
     final String PRODUCE_CONSUME_PERSISTENT_TOPIC = "persistent://" + TenantAndNsName + TestProduceConsumeTopicName;
     final String PRODUCE_CONSUME_NON_PERSISTENT_TOPIC =
                                                 "non-persistent://" + TenantAndNsName + TestProduceConsumeTopicName;
@@ -247,7 +246,6 @@ public class ResourceGroupUsageAggregationTest extends ProducerConsumerBase {
 
     // Initial set up for transport manager and producer/consumer clusters/tenants/namespaces/topics.
     private void prepareData() throws PulsarAdminException {
-        this.conf.setResourceUsageTransportPublishTopicName(INTERNAL_TOPIC);
         this.conf.setResourceUsageTransportPublishIntervalInSecs(PUBLISH_INTERVAL_SECS);
 
         this.conf.setAllowAutoTopicCreation(true);