GoogleCloudPlatform / GoogleCloudPlatform/professional-services-data-validator

PostgreSQL column validation on 38 digit number fails

Open
#1,360 3 comments 0 reactions 0 assignees View on GitHub
priority: p2
Dominant language
Python
Stars
524
Forks
171
Avg merge
5d 15h
Merged PRs (30d)
4

Description

Test case using our existing integration test table:
```
data-validation validate column -sc ora -tc pg \
--tables-list=pso_data_validator.dvt_large_decimals \
--count="*" --min="*" --max="*" --sum="*" \
--filter-status=fail
╒═══════════════════╤═══════════════════╤═══════════════════════════════════════╤══════════════════════╤════════════════════════════════════════╤═════════════════════════════════════════╤══════════════════╤═════════════════════╤══════════════════════════════════════╕
│ validation_name │ validation_type │ source_table_name │ source_column_name │ source_agg_value │ target_agg_value │ pct_difference │ validation_status │ run_id │
╞═══════════════════╪═══════════════════╪═══════════════════════════════════════╪══════════════════════╪════════════════════════════════════════╪═════════════════════════════════════════╪══════════════════╪═════════════════════╪══════════════════════════════════════╡
│ sum__col_dec_38 │ Column │ pso_data_validator.dvt_large_decimals │ col_dec_38 │ 67037036703703703670370370367037037034 │ ####################################.## │ nan │ fail │ 7aebf793-04b8-4443-958d-b53136c16fad │
╘═══════════════════╧═══════════════════╧═══════════════════════════════════════╧══════════════════════╧════════════════════════════════════════╧═════════════════════════════════════════╧══════════════════╧═════════════════════╧══════════════════════════════════════╛
```

Notice the overflowing value returned from PostgreSQL.

Search for "issue-1360" to find tests for this test table that need "col_dec_38" reinstating.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.