feat: deprecate TransferableSkus field (#44)

This PR was generated using Autosynth. 🌈 Synth log will be available here: https://source.cloud.google.com/results/invocations/679a8fed-21f3-4860-990c-bf5199c9e87f/targets - [ ] To automatically regenerate this PR, check this box. PiperOrigin-RevId: 358862009 Source-Link: googleapis/googleapis@2916ac4 BREAKING CHANGE: TransferableSkus field deprecation. Okay to make this change since no clients are using the libraries that were released just last week. The fields were supposed to be deprecated but missed. We have communicated to clients about this change.
googleapis · Feb 23, 2021 · 3acf948 · 3acf948
1 parent 8496f5e
commit 3acf948
Show file tree

Hide file tree

Showing 20 changed files with 1,336 additions and 2,326 deletions.
diff --git a/...le-cloud-channel/src/main/java/com/google/cloud/channel/v1/CloudChannelServiceClient.java b/...le-cloud-channel/src/main/java/com/google/cloud/channel/v1/CloudChannelServiceClient.java
diff --git a/...loud-channel/src/test/java/com/google/cloud/channel/v1/CloudChannelServiceClientTest.java b/...loud-channel/src/test/java/com/google/cloud/channel/v1/CloudChannelServiceClientTest.java
@@ -1577,7 +1577,6 @@ public void createChannelPartnerLinkTest() throws Exception {
         CreateChannelPartnerLinkRequest.newBuilder()
             .setParent("parent-995424086")
             .setChannelPartnerLink(ChannelPartnerLink.newBuilder().build())
-            .setDomain("domain-1326197564")
             .build();
 
     ChannelPartnerLink actualResponse = client.createChannelPartnerLink(request);
@@ -1590,7 +1589,6 @@ public void createChannelPartnerLinkTest() throws Exception {
 
     Assert.assertEquals(request.getParent(), actualRequest.getParent());
     Assert.assertEquals(request.getChannelPartnerLink(), actualRequest.getChannelPartnerLink());
-    Assert.assertEquals(request.getDomain(), actualRequest.getDomain());
     Assert.assertTrue(
         channelProvider.isHeaderSent(
             ApiClientHeaderProvider.getDefaultApiClientHeaderKey(),
@@ -1607,7 +1605,6 @@ public void createChannelPartnerLinkExceptionTest() throws Exception {
           CreateChannelPartnerLinkRequest.newBuilder()
               .setParent("parent-995424086")
               .setChannelPartnerLink(ChannelPartnerLink.newBuilder().build())
-              .setDomain("domain-1326197564")
               .build();
       client.createChannelPartnerLink(request);
       Assert.fail("No exception raised");

diff --git a/...e-cloud-channel-v1/src/main/java/com/google/cloud/channel/v1/CloudChannelServiceGrpc.java b/...e-cloud-channel-v1/src/main/java/com/google/cloud/channel/v1/CloudChannelServiceGrpc.java
diff --git a/proto-google-cloud-channel-v1/clirr-ignored-differences.xml b/proto-google-cloud-channel-v1/clirr-ignored-differences.xml
@@ -0,0 +1,54 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!-- see http://www.mojohaus.org/clirr-maven-plugin/examples/ignored-differences.html -->
+<differences>
+  <difference>
+    <className>com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest*</className>
+    <differenceType>7002</differenceType>
+    <method>* get*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest*</className>
+    <differenceType>7002</differenceType>
+    <method>* set*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest*</className>
+    <differenceType>7002</differenceType>
+    <method>* clear*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/TransferableSku*</className>
+    <differenceType>7002</differenceType>
+    <method>* get*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/TransferableSku*</className>
+    <differenceType>7002</differenceType>
+    <method>* has*(*)</method>
+  </difference>
+  <difference>
+  <className>com/google/cloud/channel/v1/TransferableSku*</className>
+  <differenceType>7002</differenceType>
+  <method>* clear*(*)</method>
+</difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/TransferableSku*</className>
+    <differenceType>7002</differenceType>
+    <method>* set*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/TransferableSku*</className>
+    <differenceType>7002</differenceType>
+    <method>* merge*(*)</method>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest*</className>
+    <differenceType>6011</differenceType>
+    <field>*_FIELD_NUMBER</field>
+  </difference>
+  <difference>
+    <className>com/google/cloud/channel/v1/TransferableSku*</className>
+    <differenceType>6011</differenceType>
+    <field>*_FIELD_NUMBER</field>
+  </difference>
+</differences>
diff --git a/...channel-v1/src/main/java/com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest.java b/...channel-v1/src/main/java/com/google/cloud/channel/v1/CreateChannelPartnerLinkRequest.java
@@ -40,7 +40,6 @@ private CreateChannelPartnerLinkRequest(
 
   private CreateChannelPartnerLinkRequest() {
     parent_ = "";
-    domain_ = "";
   }
 
   @java.lang.Override
@@ -93,13 +92,6 @@ private CreateChannelPartnerLinkRequest(
                 channelPartnerLink_ = subBuilder.buildPartial();
               }
 
-              break;
-            }
-          case 26:
-            {
-              java.lang.String s = input.readStringRequireUtf8();
-
-              domain_ = s;
               break;
             }
           default:
@@ -249,59 +241,6 @@ public com.google.cloud.channel.v1.ChannelPartnerLinkOrBuilder getChannelPartner
     return getChannelPartnerLink();
   }
 
-  public static final int DOMAIN_FIELD_NUMBER = 3;
-  private volatile java.lang.Object domain_;
-  /**
-   *
-   *
-   * <pre>
-   * Optional. The invited partner's domain. Either domain or
-   * channel_partner_link.reseller_cloud_identity_id can be used to create a
-   * link.
-   * </pre>
-   *
-   * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-   *
-   * @return The domain.
-   */
-  @java.lang.Override
-  public java.lang.String getDomain() {
-    java.lang.Object ref = domain_;
-    if (ref instanceof java.lang.String) {
-      return (java.lang.String) ref;
-    } else {
-      com.google.protobuf.ByteString bs = (com.google.protobuf.ByteString) ref;
-      java.lang.String s = bs.toStringUtf8();
-      domain_ = s;
-      return s;
-    }
-  }
-  /**
-   *
-   *
-   * <pre>
-   * Optional. The invited partner's domain. Either domain or
-   * channel_partner_link.reseller_cloud_identity_id can be used to create a
-   * link.
-   * </pre>
-   *
-   * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-   *
-   * @return The bytes for domain.
-   */
-  @java.lang.Override
-  public com.google.protobuf.ByteString getDomainBytes() {
-    java.lang.Object ref = domain_;
-    if (ref instanceof java.lang.String) {
-      com.google.protobuf.ByteString b =
-          com.google.protobuf.ByteString.copyFromUtf8((java.lang.String) ref);
-      domain_ = b;
-      return b;
-    } else {
-      return (com.google.protobuf.ByteString) ref;
-    }
-  }
-
   private byte memoizedIsInitialized = -1;
 
   @java.lang.Override
@@ -322,9 +261,6 @@ public void writeTo(com.google.protobuf.CodedOutputStream output) throws java.io
     if (channelPartnerLink_ != null) {
       output.writeMessage(2, getChannelPartnerLink());
     }
-    if (!getDomainBytes().isEmpty()) {
-      com.google.protobuf.GeneratedMessageV3.writeString(output, 3, domain_);
-    }
     unknownFields.writeTo(output);
   }
 
@@ -340,9 +276,6 @@ public int getSerializedSize() {
     if (channelPartnerLink_ != null) {
       size += com.google.protobuf.CodedOutputStream.computeMessageSize(2, getChannelPartnerLink());
     }
-    if (!getDomainBytes().isEmpty()) {
-      size += com.google.protobuf.GeneratedMessageV3.computeStringSize(3, domain_);
-    }
     size += unknownFields.getSerializedSize();
     memoizedSize = size;
     return size;
@@ -364,7 +297,6 @@ public boolean equals(final java.lang.Object obj) {
     if (hasChannelPartnerLink()) {
       if (!getChannelPartnerLink().equals(other.getChannelPartnerLink())) return false;
     }
-    if (!getDomain().equals(other.getDomain())) return false;
     if (!unknownFields.equals(other.unknownFields)) return false;
     return true;
   }
@@ -382,8 +314,6 @@ public int hashCode() {
       hash = (37 * hash) + CHANNEL_PARTNER_LINK_FIELD_NUMBER;
       hash = (53 * hash) + getChannelPartnerLink().hashCode();
     }
-    hash = (37 * hash) + DOMAIN_FIELD_NUMBER;
-    hash = (53 * hash) + getDomain().hashCode();
     hash = (29 * hash) + unknownFields.hashCode();
     memoizedHashCode = hash;
     return hash;
@@ -538,8 +468,6 @@ public Builder clear() {
         channelPartnerLink_ = null;
         channelPartnerLinkBuilder_ = null;
       }
-      domain_ = "";
-
       return this;
     }
 
@@ -573,7 +501,6 @@ public com.google.cloud.channel.v1.CreateChannelPartnerLinkRequest buildPartial(
       } else {
         result.channelPartnerLink_ = channelPartnerLinkBuilder_.build();
       }
-      result.domain_ = domain_;
       onBuilt();
       return result;
     }
@@ -631,10 +558,6 @@ public Builder mergeFrom(com.google.cloud.channel.v1.CreateChannelPartnerLinkReq
       if (other.hasChannelPartnerLink()) {
         mergeChannelPartnerLink(other.getChannelPartnerLink());
       }
-      if (!other.getDomain().isEmpty()) {
-        domain_ = other.domain_;
-        onChanged();
-      }
       this.mergeUnknownFields(other.unknownFields);
       onChanged();
       return this;
@@ -1004,122 +927,6 @@ public com.google.cloud.channel.v1.ChannelPartnerLink.Builder getChannelPartnerL
       return channelPartnerLinkBuilder_;
     }
 
-    private java.lang.Object domain_ = "";
-    /**
-     *
-     *
-     * <pre>
-     * Optional. The invited partner's domain. Either domain or
-     * channel_partner_link.reseller_cloud_identity_id can be used to create a
-     * link.
-     * </pre>
-     *
-     * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-     *
-     * @return The domain.
-     */
-    public java.lang.String getDomain() {
-      java.lang.Object ref = domain_;
-      if (!(ref instanceof java.lang.String)) {
-        com.google.protobuf.ByteString bs = (com.google.protobuf.ByteString) ref;
-        java.lang.String s = bs.toStringUtf8();
-        domain_ = s;
-        return s;
-      } else {
-        return (java.lang.String) ref;
-      }
-    }
-    /**
-     *
-     *
-     * <pre>
-     * Optional. The invited partner's domain. Either domain or
-     * channel_partner_link.reseller_cloud_identity_id can be used to create a
-     * link.
-     * </pre>
-     *
-     * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-     *
-     * @return The bytes for domain.
-     */
-    public com.google.protobuf.ByteString getDomainBytes() {
-      java.lang.Object ref = domain_;
-      if (ref instanceof String) {
-        com.google.protobuf.ByteString b =
-            com.google.protobuf.ByteString.copyFromUtf8((java.lang.String) ref);
-        domain_ = b;
-        return b;
-      } else {
-        return (com.google.protobuf.ByteString) ref;
-      }
-    }
-    /**
-     *
-     *
-     * <pre>
-     * Optional. The invited partner's domain. Either domain or
-     * channel_partner_link.reseller_cloud_identity_id can be used to create a
-     * link.
-     * </pre>
-     *
-     * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-     *
-     * @param value The domain to set.
-     * @return This builder for chaining.
-     */
-    public Builder setDomain(java.lang.String value) {
-      if (value == null) {
-        throw new NullPointerException();
-      }
-
-      domain_ = value;
-      onChanged();
-      return this;
-    }
-    /**
-     *
-     *
-     * <pre>
-     * Optional. The invited partner's domain. Either domain or
-     * channel_partner_link.reseller_cloud_identity_id can be used to create a
-     * link.
-     * </pre>
-     *
-     * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-     *
-     * @return This builder for chaining.
-     */
-    public Builder clearDomain() {
-
-      domain_ = getDefaultInstance().getDomain();
-      onChanged();
-      return this;
-    }
-    /**
-     *
-     *
-     * <pre>
-     * Optional. The invited partner's domain. Either domain or
-     * channel_partner_link.reseller_cloud_identity_id can be used to create a
-     * link.
-     * </pre>
-     *
-     * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-     *
-     * @param value The bytes for domain to set.
-     * @return This builder for chaining.
-     */
-    public Builder setDomainBytes(com.google.protobuf.ByteString value) {
-      if (value == null) {
-        throw new NullPointerException();
-      }
-      checkByteStringIsUtf8(value);
-
-      domain_ = value;
-      onChanged();
-      return this;
-    }
-
     @java.lang.Override
     public final Builder setUnknownFields(final com.google.protobuf.UnknownFieldSet unknownFields) {
       return super.setUnknownFields(unknownFields);

diff --git a/...1/src/main/java/com/google/cloud/channel/v1/CreateChannelPartnerLinkRequestOrBuilder.java b/...1/src/main/java/com/google/cloud/channel/v1/CreateChannelPartnerLinkRequestOrBuilder.java
@@ -98,33 +98,4 @@ public interface CreateChannelPartnerLinkRequestOrBuilder
    * </code>
    */
   com.google.cloud.channel.v1.ChannelPartnerLinkOrBuilder getChannelPartnerLinkOrBuilder();
-
-  /**
-   *
-   *
-   * <pre>
-   * Optional. The invited partner's domain. Either domain or
-   * channel_partner_link.reseller_cloud_identity_id can be used to create a
-   * link.
-   * </pre>
-   *
-   * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-   *
-   * @return The domain.
-   */
-  java.lang.String getDomain();
-  /**
-   *
-   *
-   * <pre>
-   * Optional. The invited partner's domain. Either domain or
-   * channel_partner_link.reseller_cloud_identity_id can be used to create a
-   * link.
-   * </pre>
-   *
-   * <code>string domain = 3 [(.google.api.field_behavior) = OPTIONAL];</code>
-   *
-   * @return The bytes for domain.
-   */
-  com.google.protobuf.ByteString getDomainBytes();
 }