[jira] [Commented] (AIRFLOW-4765) Recursion error in DataProcPigOperator

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/AIRFLOW-4765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16865581#comment-16865581 ] 

ASF GitHub Bot commented on AIRFLOW-4765:
-----------------------------------------

nuclearpinguin commented on pull request #5426: [AIRFLOW-4765] Fix DataProcPigOperator execute method
URL: https://github.com/apache/airflow/pull/5426
 
 
   Make sure you have checked _all_ steps below.
   
   ### Jira
   
   - [ ] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW/) issues and references them in the PR title. 
     - https://issues.apache.org/jira/browse/AIRFLOW-4765
   
   ### Description
   
   - [ ] Here are some details about my PR, including screenshots of any UI changes:
   This PR fixes a recursion bug in `DataProcPigOperator`.
   
   ### Tests
   
   - [ ] My PR adds unit test for `DataProcPigOperator`.
   
   ### Code Quality
   
   - [ ] Passes `flake8`
   
 
----------------------------------------------------------------
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:
users@infra.apache.org


> Recursion error in DataProcPigOperator
> --------------------------------------
>
>                 Key: AIRFLOW-4765
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-4765
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: gcp
>    Affects Versions: 1.10.3
>            Reporter: Tomasz Urbaszek
>            Assignee: Tomasz Urbaszek
>            Priority: Major
>             Fix For: 1.10.4
>
>
> Recursion error in DataProcPigOperator in execute method:
>  
> {code:java}
> class DataProcPigOperator(DataProcJobBaseOperator):
>     ...
>     def execute(self, context):
>         self.create_job_template()
>         if self.query is None:
>             self.job_template.add_query_uri(self.query_uri)
>         else:
>             self.job_template.add_query(self.query)
>             self.job_template.add_variables(self.variables)
>         self.execute(context) -> super().execute(context){code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)