MySQL/MariaDB: return max length for text columns #2133

Merged
lunny merged 4 commits from pilou/xorm:mysql_return_column_max_len into master 4 weeks ago
pilou commented 1 month ago

MySQL/MariaDB: return max length for text columns using CHARACTER_MAXIMUM_LENGTH.

Tests:

  • add an integration test: TestGetColumnsLength
  • update TestSyncTable3 since TableInfo isn't able to provide the column size
MySQL/MariaDB: return max length for text columns using `CHARACTER_MAXIMUM_LENGTH`. Tests: * add an integration test: `TestGetColumnsLength` * update `TestSyncTable3` since `TableInfo` isn't able to provide the column size
pilou added 3 commits 1 month ago
04e6719532
DBMetas is able to extract column size, TableInfo isn't
pilou force-pushed mysql_return_column_max_len from 04e6719532 to 7bbdfa76cd 1 month ago
pilou force-pushed mysql_return_column_max_len from 7bbdfa76cd to 1710100b5c 1 month ago
Poster

Updated:

  • rebased on master
  • test TestSyncTable3 updated
Updated: * rebased on master * test TestSyncTable3 updated
lunny added 1 commit 1 month ago
lunny added this to the 1.3.1 milestone 1 month ago
lunny added the
kind/enhancement
label 1 month ago
pilou force-pushed mysql_return_column_max_len from bcf55c0db0 to 55377b94b1 4 weeks ago
Poster

Updated:

  • rebased (on master)
  • make fmt applied
Updated: - rebased (on master) - `make fmt` applied
lunny added 1 commit 4 weeks ago
lunny approved these changes 4 weeks ago
lunny merged commit e1d4365667 into master 4 weeks ago
pilou deleted branch mysql_return_column_max_len 4 weeks ago

Reviewers

lunny approved these changes 4 weeks ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as e1d4365667.
Sign in to join this conversation.
Loading…
There is no content yet.