release: prepare for3.0.11 by yaronkaikov

delete commands do not return a value, still, it is possible that the
command will return a value different than OK.

In such a case, the error should be propagate to the caller via an
exception.

Fixes #65

Signed-off-by: Amnon Heiman <amnon@scylladb.com>
Message-Id: <20190618135312.2776-1-amnon@scylladb.com>
(cherry picked from commit 2fac82434b)

SCYLLA-VERSION-GEN

@@ -1,6 +1,6 @@
 #!/bin/sh
 
-VERSION=666.development
+VERSION=3.0.11
 
 if test -f version
 then

dist/debian/apt/apt.conf.d/99jessie-backports

@@ -0,0 +1 @@
+Acquire::Check-Valid-Until "false";

dist/debian/build_deb.sh

@@ -141,7 +141,7 @@ fi
 
 sudo rm -fv /var/cache/pbuilder/$PRODUCT-jmx-$TARGET.tgz
 sudo PRODUCT=$PRODUCT DIST=$TARGET /usr/sbin/pbuilder clean --configfile ./dist/debian/pbuilderrc
-sudo PRODUCT=$PRODUCT DIST=$TARGET /usr/sbin/pbuilder create --configfile ./dist/debian/pbuilderrc
+sudo PRODUCT=$PRODUCT DIST=$TARGET /usr/sbin/pbuilder create --configfile ./dist/debian/pbuilderrc --aptconfdir dist/debian/apt
 sudo PRODUCT=$PRODUCT DIST=$TARGET /usr/sbin/pbuilder update --configfile ./dist/debian/pbuilderrc
 if [ "$TARGET" = "jessie" ]; then
     echo "apt-get install -y -t jessie-backports ca-certificates-java" > build/jessie-pkginst.sh

dist/debian/pbuilderrc

@@ -18,7 +18,7 @@ elif [ "$DIST" = "jessie" ] || [ "$DIST" = "stretch" ] || [ "$DIST" = "buster" ]
     COMPONENTS="main contrib non-free"
     DEBOOTSTRAPOPTS="--keyring=/usr/share/keyrings/debian-archive-keyring.gpg"
     if [ "$DIST" = "jessie" ]; then
-        OTHERMIRROR="deb http://httpredir.debian.org/debian jessie-backports main"
+        OTHERMIRROR="deb http://archive.debian.org/debian jessie-backports main"
     fi
 else
     echo "Unknown distribution: $DIST"

src/main/java/com/scylladb/jmx/api/APIClient.java

@@ -152,7 +152,11 @@ public class APIClient {
             get(path, queryParams).delete();
             return;
         }
-        get(path).delete();
+        Response response = get(path).delete();
+        if (response.getStatus() != Response.Status.OK.getStatusCode()) {
+            throw getException("Scylla API server HTTP delete to URL '" + path + "' failed",
+                    response.readEntity(String.class));
+        }
     }
 
     public void delete(String path) {

src/main/java/com/scylladb/jmx/utils/APIMBeanServer.java

@@ -53,7 +53,16 @@ public class APIMBeanServer implements MBeanServer {
     }
 
     private static ObjectName prepareForRemote(final ObjectName n) {
-        return ObjectName.getInstance(n);
+        /*
+         * ObjectName.getInstance has changed in JDK (micro) updates so it no longer applies 
+         * overridable methods -> wrong name published. 
+         * Fix by doing explicit ObjectName instansiation. 
+         */
+        try {
+            return new ObjectName(n.getCanonicalName());
+        } catch (MalformedObjectNameException e) {
+            throw new IllegalArgumentException(n.toString());
+        }
     }
 
     @Override

src/main/java/org/apache/cassandra/metrics/TableMetrics.java

@@ -297,17 +297,16 @@ public class TableMetrics implements Metrics {
 
     @SuppressWarnings("serial")
     static class TableMetricObjectName extends javax.management.ObjectName {
-        private static final String FAKE_NAME = "a:a=a";
-
         private final TableMetricStringNameFactory factory;
         private final String metricName;
 
         public TableMetricObjectName(TableMetricStringNameFactory factory, String metricName) throws MalformedObjectNameException {
-            super(FAKE_NAME);
+            super("");
             this.factory = factory;
             this.metricName = metricName;
         }
 
+        
         @Override
         public boolean isPropertyValuePattern(String property) {
             return false;
@@ -356,8 +355,7 @@ public class TableMetrics implements Metrics {
         @Override
         public boolean equals(Object o) {
             if (this == o) return true;
-            if (!(o instanceof TableMetricObjectName)) return false;
-            return getCanonicalName().equals(((TableMetricObjectName) o).getCanonicalName());
+            return getCanonicalName().equals(((ObjectName) o).getCanonicalName());
         }
 
         @Override
@@ -372,6 +370,36 @@ public class TableMetrics implements Metrics {
             }
             return getCanonicalName().equals(name.getCanonicalName());
         }
+
+
+        @Override
+        public boolean isPattern() {
+            return false;
+        }
+
+
+        @Override
+        public boolean isDomainPattern() {
+            return false;
+        }
+
+
+        @Override
+        public boolean isPropertyPattern() {
+            return false;
+        }
+
+
+        @Override
+        public boolean isPropertyListPattern() {
+            return false;
+        }
+
+
+        @Override
+        public boolean isPropertyValuePattern() {
+            return false;
+        }
     }
 
     static interface TableMetricStringNameFactory {