turbaszek commented on a change in pull request #7743: [AIRFLOW-7075] Operators 
for storing information from GCS into GA
URL: https://github.com/apache/airflow/pull/7743#discussion_r397709240
 
 

 ##########
 File path: 
airflow/providers/google/marketing_platform/example_dags/example_analytics.py
 ##########
 @@ -21,16 +21,21 @@
 
 from airflow import models
 from airflow.providers.google.marketing_platform.operators.analytics import (
+    GoogleAnalyticsDataImportUploadOperator, 
GoogleAnalyticsDeletePreviousDataUploadsOperator,
     GoogleAnalyticsGetAdsLinkOperator, GoogleAnalyticsListAccountsOperator,
-    GoogleAnalyticsRetrieveAdsLinksListOperator,
+    GoogleAnalyticsModifyFileHeadersDataImportOperator, 
GoogleAnalyticsRetrieveAdsLinksListOperator,
 )
 from airflow.utils import dates
 
 ACCOUNT_ID = os.environ.get("GA_ACCOUNT_ID", "123456789")
 
 Review comment:
   Yes, in case someone run this DAG without env variable the error will say 
something about "account id not found" instead of "account_id wrong type" 
(default would be `None`).

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to