[break] Rename to Conjure ecosystem (#117)

Author: iamdanfox

build.gradle

@@ -34,7 +34,7 @@ dependencies {
 
 allprojects {
     version gitVersion()
-    group 'com.palantir.remoting-api'
+    group 'com.palantir.conjure.java.api'
 
     apply plugin: 'com.palantir.configuration-resolver'
     apply plugin: 'nebula.dependency-recommender'

errors/build.gradle

@@ -7,7 +7,7 @@ dependencies {
     compile "com.google.code.findbugs:jsr305"
     compile "com.palantir.safe-logging:safe-logging"
     compile "javax.ws.rs:javax.ws.rs-api"
-    compile "com.palantir.conjure.java.api:errors"
+    implementation "com.palantir.safe-logging:preconditions"
 
     testCompile project(":extras:jackson-support")
     testCompile "org.assertj:assertj-core"

errors/src/main/java/com/palantir/remoting/api/errors/ErrorType.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/ErrorType.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
@@ -52,33 +52,6 @@ public enum Code {
         Code(Integer httpErrorCode) {
             this.httpErrorCode = httpErrorCode;
         }
-
-        @SuppressWarnings("checkstyle:CyclomaticComplexity")
-        public com.palantir.conjure.java.api.errors.ErrorType.Code asConjure() {
-            switch (this) {
-                case PERMISSION_DENIED:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.PERMISSION_DENIED;
-                case INVALID_ARGUMENT:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.INVALID_ARGUMENT;
-                case NOT_FOUND:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.NOT_FOUND;
-                case CONFLICT:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.CONFLICT;
-                case REQUEST_ENTITY_TOO_LARGE:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.REQUEST_ENTITY_TOO_LARGE;
-                case FAILED_PRECONDITION:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.FAILED_PRECONDITION;
-                case INTERNAL:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.INTERNAL;
-                case TIMEOUT:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.TIMEOUT;
-                case CUSTOM_CLIENT:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.CUSTOM_CLIENT;
-                case CUSTOM_SERVER:
-                    return com.palantir.conjure.java.api.errors.ErrorType.Code.CUSTOM_SERVER;
-            }
-            throw new UnsupportedOperationException("Unable to convert to Conjure ErrorType.Code");
-        }
     }
 
     public static final ErrorType PERMISSION_DENIED =
@@ -106,10 +79,6 @@ public com.palantir.conjure.java.api.errors.ErrorType.Code asConjure() {
     /** The HTTP error code used to convey this error to HTTP clients. */
     public abstract int httpErrorCode();
 
-    public final com.palantir.conjure.java.api.errors.ErrorType asConjure() {
-        return com.palantir.conjure.java.api.errors.ErrorType.create(code().asConjure(), name());
-    }
-
     @Value.Check
     final void check() {
         if (!ERROR_NAME_PATTERN.matcher(name()).matches()) {
@@ -118,26 +87,6 @@ final void check() {
         }
     }
 
-    /**
-     * Creates a new error type with code {@link Code#CUSTOM_CLIENT} and the given name.
-     *
-     * @deprecated Use {@link #create}. This method will be removed in version 2.0.0.
-     */
-    @Deprecated
-    public static ErrorType client(String name) {
-        return createAndCheckNamespaceIsNotDefault(Code.CUSTOM_CLIENT, name);
-    }
-
-    /**
-     * Creates a new error type with code {@link Code#CUSTOM_SERVER} and the given name.
-     *
-     * @deprecated Use {@link #create}. This method will be removed in version 2.0.0.
-     */
-    @Deprecated
-    public static ErrorType server(String name) {
-        return createAndCheckNamespaceIsNotDefault(Code.CUSTOM_SERVER, name);
-    }
-
     /**
      * Constructs an {@link ErrorType} with the given error {@link Code} and name.
      */

ssl-config/src/main/java/com/palantir/remoting/api/config/ssl/ImmutablesStyle.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/ImmutablesStyle.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.config.ssl;
+package com.palantir.conjure.java.api.errors;
 
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;

errors/src/main/java/com/palantir/remoting/api/errors/QosException.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/QosException.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 import com.palantir.logsafe.Arg;
 import com.palantir.logsafe.SafeArg;

errors/src/main/java/com/palantir/remoting/api/errors/RemoteException.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/RemoteException.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 /**
  * An exception thrown by an RPC client to indicate remote/server-side failure.
@@ -44,9 +44,4 @@ public RemoteException(SerializableError error, int status) {
         this.error = error;
         this.status = status;
     }
-
-    /** Converts this exception to an equivalent conjure-java-api RemoteException. */
-    public com.palantir.conjure.java.api.errors.RemoteException asConjure() {
-        return new com.palantir.conjure.java.api.errors.RemoteException(error.asConjure(), status);
-    }
 }

errors/src/main/java/com/palantir/remoting/api/errors/SerializableError.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/SerializableError.java

@@ -14,13 +14,14 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
 import com.palantir.logsafe.Arg;
+import com.palantir.logsafe.exceptions.SafeIllegalStateException;
 import java.io.Serializable;
 import java.util.Map;
 import java.util.Objects;
@@ -48,7 +49,7 @@ public abstract class SerializableError implements Serializable {
     @JsonProperty("errorCode")
     @Value.Default
     public String errorCode() {
-        return getExceptionClass().orElseThrow(() -> new IllegalStateException(
+        return getExceptionClass().orElseThrow(() -> new SafeIllegalStateException(
                 "Expected either 'errorCode' or 'exceptionClass' to be set"));
     }
 
@@ -60,7 +61,7 @@ public String errorCode() {
     @JsonProperty("errorName")
     @Value.Default
     public String errorName() {
-        return getMessage().orElseThrow(() -> new IllegalStateException(
+        return getMessage().orElseThrow(() -> new SafeIllegalStateException(
                 "Expected either 'errorName' or 'message' to be set"));
     }
 
@@ -115,24 +116,6 @@ public static SerializableError forException(ServiceException exception) {
         return builder.build();
     }
 
-    public final com.palantir.conjure.java.api.errors.SerializableError asConjure() {
-        com.palantir.conjure.java.api.errors.SerializableError.Builder builder =
-                com.palantir.conjure.java.api.errors.SerializableError.builder();
-
-        if (!getExceptionClass().isPresent()) {
-            builder.errorCode(errorCode());
-            builder.errorName(errorName());
-        } else {
-            // this is for back-compat, old remoting2 exceptions have 'exceptionClass' and 'message' fields.
-            builder.exceptionClass(getExceptionClass().get());
-            builder.message(getMessage().get());
-        }
-
-        builder.errorInstanceId(errorInstanceId());
-
-        return builder.build();
-    }
-
     // TODO(rfink): Remove once all error producers have switched to errorCode/errorName.
     public static final class Builder extends ImmutableSerializableError.Builder {}
 

errors/src/main/java/com/palantir/remoting/api/errors/ServiceException.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/ServiceException.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 import com.palantir.logsafe.Arg;
 import com.palantir.logsafe.SafeArg;

errors/src/main/java/com/palantir/remoting/api/errors/UnknownRemoteException.java renamed to errors/src/main/java/com/palantir/conjure/java/api/errors/UnknownRemoteException.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 /**
  * An exception thrown by an RPC client to indicate remote/server-side failure from a non-remoting server.

errors/src/test/java/com/palantir/remoting/api/errors/ErrorTypeTest.java renamed to errors/src/test/java/com/palantir/conjure/java/api/errors/ErrorTypeTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.palantir.remoting.api.errors;
+package com.palantir.conjure.java.api.errors;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;