impl new derive macro to replace attribute macro

Kilerd commented 1 week ago

this PR is to impl issue #58

main change:

remove #[tonbo_record] macro, use #[derive(Record)] instead
remove #[primary_key] attribute, use #[record(primary_key)]
remove #[tonbo_record(::serde::Serialize)] logic, use #[derive(Record, ::serde::Serialize)] instead
remove builtin trait derive impl ::tonbo::KeyAttributes, Debug, PartialEq, Eq, Clone

refactor related:

separate record code gen into small handler for human-friendly maintenance
extract ident generator util for Centralized Management

further optimization:

detect multiple primary_key definition (had added todo item in codebase)
rename generated struct's name for code isolation, like we will generate struct UserRef for struct User, which may cause code conflict if use had defined one in his codebase. the common pattern is to add unique prefix for generated struct , E.G. __TonboUserRef
possible to remove trait Debug as required trait for trait Record, cuz it will slow down the compile time. and have no idea why we need this trait.

Kilerd commented 1 week ago

and BTW I dont use tonbo in any of my projects, if you guys can provide some testcases are very helpful.

KKould commented 1 week ago

and BTW I dont use tonbo in any of my projects, if you guys can provide some testcases are very helpful.

This is a very big and meaningful PR, I will continue to follow up on the testing work

Kilerd commented 1 week ago

yep, you're right, and I recommend to review this PR commit by commit.

KKould commented 1 week ago

@Kilerd @ethe at present, macros does not support many configurable functions. Its core is to help users reduce code, so I only add tests to macro to verify the correctness. please take a look

https://github.com/tonbo-io/tonbo/pull/141/commits/7f7cb395ac4b6c2b0e5a85c4aedce45a42dee07c

ethe commented 1 week ago

This is pretty neat! Thanks to @Kilerd

tonbo-io / tonbo

impl new derive macro to replace attribute macro #141