[ 
https://issues.apache.org/jira/browse/BEAM-14326?focusedWorklogId=760142&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-760142
 ]

ASF GitHub Bot logged work on BEAM-14326:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 21/Apr/22 15:50
            Start Date: 21/Apr/22 15:50
    Worklog Time Spent: 10m 
      Work Description: reuvenlax commented on code in PR #17428:
URL: https://github.com/apache/beam/pull/17428#discussion_r855337154


##########
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/StorageApiDynamicDestinationsTableRow.java:
##########
@@ -95,7 +96,14 @@ public MessageConverter<T> getMessageConverter(
                       + "using a create disposition of CREATE_IF_NEEDED.");
             }
           }
+        } else {
+          // Make sure we register this schema with the cache, unless there's 
already a more
+          // up-to-date schema.
+          tableSchema =
+              MoreObjects.firstNonNull(
+                  SCHEMA_CACHE.putSchemaIfAbsent(tableReference, tableSchema), 
tableSchema);

Review Comment:
   Yes - mostly applies to unit tests (where we explicitly clear the static 
caches in between tests) - I don't think this case would occur in in production.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 760142)
    Time Spent: 40m  (was: 0.5h)

> Pipelines using BigQueryIO connector with one of the Storage Write API 
> methods don't exit the launch process
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: BEAM-14326
>                 URL: https://issues.apache.org/jira/browse/BEAM-14326
>             Project: Beam
>          Issue Type: Bug
>          Components: io-java-gcp
>            Reporter: Sergei Lilichenko
>            Assignee: Reuven Lax
>            Priority: P1
>             Fix For: 2.39.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> The pipeline starts, but the launch process doesn't exit.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to