Skip to content

Minor refactors to SqlServer type mapping tests#1

Closed
tangjiangling wants to merge 1 commit intomasterfrom
minor-fixes-in-sql-server-type-mapping-tests
Closed

Minor refactors to SqlServer type mapping tests#1
tangjiangling wants to merge 1 commit intomasterfrom
minor-fixes-in-sql-server-type-mapping-tests

Conversation

@tangjiangling
Copy link
Copy Markdown
Owner

@tangjiangling tangjiangling commented Feb 26, 2022

  • Move sqlServer to BaseSqlServerTypeMapping
  • Remove onRemoteDatabase in TestSqlServerTypeMapping
  • Implement onRemoteDatabase in BaseSqlServerTypeMapping

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Refactoring.

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

SqlServer connector type mapping tests.

How would you describe this change to a non-technical end user or system administrator?

N/A.

Related issues, pull requests, and links

trinodb#11148 (comment)

Documentation

( ) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

- Move `sqlServer` to BaseSqlServerTypeMapping
- Remove onRemoteDatabase in TestSqlServerTypeMapping
- Implement onRemoteDatabase in BaseSqlServerTypeMapping
@tangjiangling tangjiangling deleted the minor-fixes-in-sql-server-type-mapping-tests branch February 26, 2022 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant