GoogleCloudPlatform / GoogleCloudPlatform/professional-services-data-validator

Deprecated -tsp option appears to suppress further logging

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

Description

**Describe the bug**
Usage of deprecated `-tsp` option appears to suppress further logging.

**What version of DVT are you using?**
7.7.0

**Steps to reproduce the behavior**
1. Command without `-tsp`, notice the INFO logging:

```
$ data-validation validate row -sc ora -tc pg \
-tbls=pso_data_validator.dvt_core_types --hash="*" \
-rh=pg_rh.pso_data_validator_results.results
07/14/2025 03:06:15 PM-INFO: {"run_id": "9c0e7e2c-31dd-4608-84ce-69cb3f8b4468", "start_time": "2025-07-14T15:06:14.533401+00:00", "end_time": "2025-07-14T15:06:15.487981+00:00", "total_source_rows": 3, "total_target_rows": 3, "total_rows_validated": 3, "total_rows_success_validation_status": 0, "total_rows_fail_validation_status": 3, "failed_rows_present_in_source_not_in_target": 0, "failed_rows_present_in_target_not_in_source": 0, "failed_rows_present_in_both_source_and_target": 3}
07/14/2025 03:06:15 PM-INFO: Results written to pso_data_validator_results.results, run id: 9c0e7e2c-31dd-4608-84ce-69cb3f8b4468
$
```

2. Same command but with `-tsp`, notice the INFO logging:

```
$ data-validation validate row -sc ora -tc pg \
-tbls=pso_data_validator.dvt_core_types --hash="*" \
-rh=pg_rh.pso_data_validator_results.results -tsp
WARNING:root:Argument -tsp is deprecated and may be removed in a future release
$
```

**Expected behavior**
The warning message should not affect logging.

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.