Support Vec<u8> primary key

[billy1624]

• Resolve Support for BYTEA Postgres primary keys #286

[billy1624]

Hey @tyt2y3, I was wondering why ColumnType::Binary is defined as without length in sea-orm?

Because of that, I can't seems to get around this.

Error: Exec("error returned from database: 1170 (42000): BLOB/TEXT column 'id' used in key specification without a key length")

[tyt2y3]

The BYTEA type in Postgres does not have a length.

[tyt2y3]

@robot-head can you share your schema definition? I bet the index should have a key length

[billy1624]

Hey @tyt2y3, I was wondering why ColumnType::Binary is defined as without length in sea-orm?

Because of that, I can't seems to get around this.

Error: Exec("error returned from database: 1170 (42000): BLOB/TEXT column 'id' used in key specification without a key length")

Btw... this only failed on MySQL & MariaDB. Postgres is working perfectly

[tyt2y3]

Okay, then we should use sea_query::ColumnType::Binary(Some(1024)) explicitly for the test case.

[billy1624]

Well, can we default 1024 it inside SeaORM?

For MySQL & MariaDB only and leave Postgres untouched?

[tyt2y3]

Edited

[billy1624]

Okay, then we should use sea_query::ColumnType::Binary(Some(1024)) explicitly for the test case.

That would work but how about ordinary user that want to use create_table_from_entity on MySQL & MariaDB? They will fall into this trap as well if they happen to use Vec<u8> (blob db type) as primary key.

[tyt2y3]

Looks good. Anything missing? We are about to make a minor release.

[billy1624]

Go for it