Artem Fomiuk
Artem Fomiuk
Could you review again please? I updated everything, should be ok
we can't get the signature right now, because this person is not working with us now, so I opened [another pull request](https://github.com/paritytech/substrate/pull/12695) with all these changes.
CI is satisfied there, just need some approvals again
Although the PR is pretty big, here are some main points for reviewing(because PR is big mostly because of renaming/import changes): * `db_connection` crate - most of the logic moved...
Also a few points worth discussion: * Right now dal crates are reexporting some types/traits from db_connection crate - it was done mostly because most of the crates are not...
The problem with merges is that I need to fix it every few hours, because PR itself touches a lot of places and almost every merge to main causes conflicts....
Do we need to fully ignore Transfer events? e.g. they appear and impact log indexes, is it okay to be this way or it is better to somehow change this...
I don't know to be honest. The discussion about it was brought up again not that long ago, but I don't remember any decision was made. Maybe @RomanBrodetski or @StanislavBreadless...
For now I want to measure just raw bounds, hardcode them to https://github.com/matter-labs/zksync-era/pull/1076 as constants and most likely will add test in the PR as well