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

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

                Author: ASF GitHub Bot
            Created on: 23/Sep/20 22:00
            Start Date: 23/Sep/20 22:00
    Worklog Time Spent: 10m 
      Work Description: pabloem commented on a change in pull request #12782:
URL: https://github.com/apache/beam/pull/12782#discussion_r493919948



##########
File path: sdks/python/apache_beam/runners/dataflow/dataflow_runner_test.py
##########
@@ -323,7 +316,9 @@ def test_no_group_by_key_directly_after_bigquery(self):
                          options=PipelineOptions(self.default_properties)) as 
p:
         # pylint: disable=expression-not-assigned
         p | beam.io.Read(
-            beam.io.BigQuerySource('dataset.faketable')) | beam.GroupByKey()
+            beam.io.BigQuerySource(
+                'dataset.faketable',
+                use_dataflow_native_source=True)) | beam.GroupByKey()

Review comment:
       for these two tests in dataflow_runner_test, we do not have equivalents 
for the custom source. This is because these failures do not occur under the 
custom source:
   - Custom source is acceptable in streaming because it becomes an SDF that 
runs fine in streaming
   - Custom source will not error out on type hints with GBK because the type 
hints are not set up to fail for this sort of composite transform. I don't know 
why, but we can file a bug to ensure this is re-enabled.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


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

    Worklog Id:     (was: 489865)
    Time Spent: 1.5h  (was: 1h 20m)

> Override Dataflow-native implementation of BQSource with a Beam source
> ----------------------------------------------------------------------
>
>                 Key: BEAM-10950
>                 URL: https://issues.apache.org/jira/browse/BEAM-10950
>             Project: Beam
>          Issue Type: Improvement
>          Components: io-py-gcp
>            Reporter: Pablo Estrada
>            Assignee: Pablo Estrada
>            Priority: P2
>             Fix For: 2.25.0
>
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to