Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BQ views Profiler; Errors on system metric computation #17681

Closed
TeddyCr opened this issue Sep 3, 2024 · 3 comments
Closed

BQ views Profiler; Errors on system metric computation #17681

TeddyCr opened this issue Sep 3, 2024 · 3 comments
Assignees
Labels
bug Something isn't working profiler

Comments

@TeddyCr
Copy link
Contributor

TeddyCr commented Sep 3, 2024

Affected module
openmetadata-ingestion

Describe the bug
View type in BQ throws the below error when computing system metrics

[2024-09-02T09:42:18.439+0000] {status.py:77} DEBUG - Traceback (most recent call last):
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/interface/sqlalchemy/profiler_interface.py", line 382, in _compute_system_metrics
    rows = metrics().sql(session, conn_config=self.service_connection_config)
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/metrics/system/system.py", line 462, in sql
    system_metrics = get_system_metrics_for_dialect(
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/utils/dispatch.py", line 99, in wrapper
    return dispatch(str(args[0]))(*args, **kwargs)
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/metrics/system/system.py", line 140, in _
    jobs = [
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/metrics/system/system.py", line 141, in <listcomp>
    BigQueryQueryResult(
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for BigQueryQueryResult
dml_statistics
  none is not an allowed value (type=type_error.none.not_allowed)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/interface/sqlalchemy/profiler_interface.py", line 446, in compute_metrics_in_thread
    row = self._get_metric_fn[metric_func.metric_type.value](
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/interface/sqlalchemy/profiler_interface.py", line 386, in _compute_system_metrics
    handle_query_exception(msg, exc, session)
  File "/home/airflow/.local/lib/python3.10/site-packages/metadata/profiler/interface/sqlalchemy/profiler_interface.py", line 62, in handle_query_exception
    raise RuntimeError(exc)
RuntimeError: 1 validation error for BigQueryQueryResult
dml_statistics
  none is not an allowed value (type=type_error.none.not_allowed)

To Reproduce
Run profiler for a BigQuery view -- system metric should throw the above error.

Version:

  • OS: [e.g. iOS]
  • Python version:
  • OpenMetadata version: [e.g. 0.8]
  • OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

@TeddyCr TeddyCr added bug Something isn't working profiler labels Sep 3, 2024
@Cir02
Copy link

Cir02 commented Nov 8, 2024

Hi, any updates on this issue?

thx!

@ayush-shah
Copy link
Member

Hello @Cir02 this has been resolved in the upcoming 1.6.0 release, the above should not be reproducible post the upgrade once it is out, it should be published by late November or early december.
Fixed with #18322
Thanks

@Cir02
Copy link

Cir02 commented Nov 8, 2024

Thanks ayush-shah !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working profiler
Projects
None yet
Development

No branches or pull requests

3 participants