feat(jobs): add database_fork job type - #200
Conversation
|
|
||
| ### Changed | ||
|
|
||
| - feat(jobs): add database_fork job type |
There was a problem hiding this comment.
nit: Add a ### Removed section with Breaking: entries for the public API that this regeneration deletes (not blocking).
The regeneration removes ConnectionsApi.purge_connection_cache, ConnectionsApi.purge_table_cache, and CreateConnectionRequest.skip_discovery. Callers of these symbols get AttributeError or a validation error after upgrade. The [0.11.0] section lists removals like these as Breaking: entries.
The single entry also names only database_fork. Consider adding the new surface too: get_database_lineage, lookup_database_by_name, set_*_constant_per_key, if_not_exists, vector_precision, and JobType.TABLE_CONSTANTS_UPDATE.
|
|
||
|
|
||
| @validate_call | ||
| def purge_connection_cache( |
There was a problem hiding this comment.
nit: Remove the purge_connection_cache call at tests/integration/test_connections_read.py:31-32, and update the module docstring there (not blocking).
This PR deletes purge_connection_cache. The test is skipped today, so CI stays green. When the seeded connection is restored and the skip is lifted, the test fails with AttributeError.
Auto-generated from the updated HotData OpenAPI spec.
Source: https://github.com/hotdata-dev/www/pull/433