Throw an error if min_valid_version is null

[luandy64]

Description of change

In logical replication, sometimes our query here returns nil

tap-mssql/src/tap_mssql/sync_strategies/logical.clj

Lines 95 to 99 in 1bfc58d

	sql-query (format "SELECT CHANGE_TRACKING_MIN_VALID_VERSION(%d) as min_valid_version" object-id)
	_ (log/infof "Executing query: %s" sql-query)
	min-valid-version (-> (jdbc/query (assoc (config/->conn-map config) :dbname dbname) [sql-query])
	first
	:min_valid_version)]

And when that happens, the tap will eventually try (> nil 12345) which causes a java.lang.NullPointerException.

QA steps

• automated tests passing
• manual qa steps passing (list below)
  • Ran the tap

Risks

• Low

Rollback steps

• revert this branch

[dmosorast]

Just one comment re: table name matching for getting object_id. There is an Object_id function as well, but Kyle says that may not have been working? https://docs.microsoft.com/en-us/sql/t-sql/functions/object-id-transact-sql?view=sql-server-ver15

Also, are there tests for this already? Maybe adding one that creates 2 different schemas could be useful.