Yiling-J
Yiling-J
@nlachfr I think approximate is good enough, but the method name should show that clearly, something like `EstimatedSize`. And to get that value, you can sum the `qlen` in all...
@nlachfr Looks good, Thanks for the contribution🙏
@nlachfr As you can see in the Len method, it simply loops through all shards and counts the items in each shard map. Adding a Size method is also straightforward;...
@idc77 Usually in tests, you only need to truncate all tables after each test. **OR**, if you run tests parallel and access same table, wrap your test ent client in...
@HurSungYun One thing I try to understand in your snippet, why can't you just get user ID from your `CreateUser`? ```go if user, err := CreateUser(c, record); err != nil...
Hi @l3pp4rd , I can help maintain this library. I'm the author of [Theine](https://github.com/Yiling-J/theine-go) and use Go a lot. Though I don't use [go-txdb](https://github.com/DATA-DOG/go-txdb) much, after taking a look the...
@stefafafan I just creating a PR and do some refactors, would you like to take a look? https://github.com/DATA-DOG/go-txdb/pull/51
@stefafafan Are you collaborator now? Do you want to take a look https://github.com/DATA-DOG/go-txdb/issues/49?
I skipped the failed test on the Windows platform. This is consistent with how Go handles form cleanup tests (https://github.com/golang/go/blob/6da16013ba4444e0d71540f68279f0283a92d05d/src/net/http/serve_test.go#L7028), so I think it’s fine.
> are you still willing to handle this issue @flc1125 Yes. But first, let’s reach an agreement on how we should fix it. As I proposed a while ago, I...