turbaszek commented on a change in pull request #11113:
URL: https://github.com/apache/airflow/pull/11113#discussion_r494165469
##########
File path: airflow/providers/google/cloud/operators/bigquery.py
##########
@@ -20,33 +20,52 @@
"""
This module contains Google BigQuery operators.
"""
+import datetime
import enum
import hashlib
import json
+import logging
+import random
import re
+import string
import uuid
import warnings
from typing import Any, Dict, Iterable, List, Optional, Sequence, Set,
SupportsAbs, Union
+from urllib.parse import urlsplit
import attr
from google.api_core.exceptions import Conflict
from google.cloud.bigquery import TableReference
+from great_expectations.data_context.types.base import DataContextConfig
Review comment:
I'm wondering how we should approach it. GE should not be a requirement
for other Google operators, on the other hand having separate package for GE
doesn't sounds like a good way. Unless we make this operator generic and
provider neutral
----------------------------------------------------------------
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]
