Abacn commented on code in PR #34525:
URL: https://github.com/apache/beam/pull/34525#discussion_r2070276364


##########
sdks/python/apache_beam/transforms/managed_iceberg_it_test.py:
##########
@@ -72,7 +72,10 @@ def test_write_read_pipeline(self):
     with beam.Pipeline(argv=self.args) as read_pipeline:
       output_dicts = (
           read_pipeline
-          | beam.managed.Read(beam.managed.ICEBERG, config=iceberg_config)
+          | beam.managed.Read(

Review Comment:
   This caused test failure: https://github.com/apache/beam/runs/41441259688
   
   RuntimeError: A transform with label "Managed Read(ICEBERG)" already exists 
in the pipeline. To apply a transform with a specified label, write pvalue | 
"label" >> transform or use the option "auto_unique_labels" to automatically 
generate unique transform labels. Note "auto_unique_labels" could cause data 
loss when updating a pipeline or reloading the job state. This is not 
recommended for streaming jobs.
   self = <apache_beam.transforms.managed_iceberg_it_test.ManagedIcebergIT 
testMethod=test_write_read_pipeline>
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to