fix: rollback the reconnect after 10MB behavior after the fix for omg… (#1530)

yirutang · web-flow · commit 737bd0e2c213 · 2022-02-11T13:17:13.000-05:00
* fix: rollback the reconnect after 10MB behavior after the fix for omg is deployed

* .
diff --git a/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/JsonStreamWriter.java b/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/JsonStreamWriter.java
@@ -405,15 +405,16 @@ public Builder setIgnoreUnknownFields(boolean ignoreUnknownFields) {
     }
 
     /**
-     * Setter for a reconnectAfter10M, temporaily workaround for omg/48020. Fix for the omg is
-     * supposed to roll out by 2/11/2022 Friday. If you set this to True, your write will be slower
-     * (0.75MB/s per connection), but your writes will not be stuck as a sympton of omg/48020.
+     * @Deprecated Setter for a reconnectAfter10M, temporaily workaround for omg/48020. Fix for the
+     * omg is supposed to roll out by 2/11/2022 Friday. If you set this to True, your write will be
+     * slower (0.75MB/s per connection), but your writes will not be stuck as a sympton of
+     * omg/48020.
      *
      * @param reconnectAfter10M
      * @return Builder
      */
     public Builder setReconnectAfter10M(boolean reconnectAfter10M) {
-      this.reconnectAfter10M = reconnectAfter10M;
+      this.reconnectAfter10M = false;
       return this;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -405,15 +405,16 @@ public Builder setIgnoreUnknownFields(boolean ignoreUnknownFields) {`
`405`	`405`	`}`
`406`	`406`
`407`	`407`	`/**`
`408`		`- * Setter for a reconnectAfter10M, temporaily workaround for omg/48020. Fix for the omg is`
`409`		`- * supposed to roll out by 2/11/2022 Friday. If you set this to True, your write will be slower`
`410`		`- * (0.75MB/s per connection), but your writes will not be stuck as a sympton of omg/48020.`
	`408`	`+ * @Deprecated Setter for a reconnectAfter10M, temporaily workaround for omg/48020. Fix for the`
	`409`	`+ * omg is supposed to roll out by 2/11/2022 Friday. If you set this to True, your write will be`
	`410`	`+ * slower (0.75MB/s per connection), but your writes will not be stuck as a sympton of`
	`411`	`+ * omg/48020.`
`411`	`412`	`*`
`412`	`413`	`* @param reconnectAfter10M`
`413`	`414`	`* @return Builder`
`414`	`415`	`*/`
`415`	`416`	`public Builder setReconnectAfter10M(boolean reconnectAfter10M) {`
`416`		`- this.reconnectAfter10M = reconnectAfter10M;`
	`417`	`+ this.reconnectAfter10M = false;`
`417`	`418`	`return this;`
`418`	`419`	`}`
`419`	`420`