Support for BYTEA Postgres primary keys

[robot-head]

Currently any column marked as a primary key needs to satisfy TryFromU64.

A BYTEA (Vec<u8>) does not satisfy this (and, I don't think ever could).

Would it be possible to add support for this type of primary key? We use it a lot in our application

[tyt2y3]

TryFromU64 can definitely return Err in that case. That's how UUID primary key is supported.
We can do something to support Vec<u8> in our test suite, but it should be working already.

sea-orm/tests/common/features/metadata.rs

Lines 5 to 15 in 57c2553

	pub struct Model {
	#[sea_orm(primary_key, auto_increment = false)]
	pub uuid: Uuid,
	#[sea_orm(column_name = "type", enum_name = "Type")]
	pub ty: String,
	pub key: String,
	pub value: String,
	pub bytes: Vec<u8>,
	pub date: Option<Date>,
	pub time: Option<Time>,
	}

[robot-head]

Seeing this

With this example model:

#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "foobar")]
pub struct Model {
    #[sea_orm(primary_key, auto_increment = false)]
    pub primary_a: Vec<u8>,
    #[sea_orm(primary_key, auto_increment = false)]
    pub primary_b: Vec<u8>,
}

[billy1624]

The errors are expected. I will create a PR now to make it work! Stay tuned

[robot-head]

Thank you so much!

Loving this crate so far btw, refactoring our large codebase to make it finally unit testable.

Thanks for the quick help

[billy1624]

Thanks for the support! Do recommend sea-orm to your friends loll :P

[billy1624]

Hey @robot-head, please try #287.

Simply...

sea-orm = { version = "^0.3", git = "https://github.com/SeaQL/sea-orm.git", branch = "vec-u8-primary-key", ... }

P.S. For now, it works on Postgres but not MySQL and MariaDB

[tyt2y3]

Loving this crate so far btw, refactoring our large codebase to make it finally unit testable

Cool, we definitely would like to know a bit more about your scenario!

[tyt2y3]

0.3.2