Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PopulateLogTile error in MySQL storage tests #3657

Open
robstradling opened this issue Oct 28, 2024 · 2 comments
Open

PopulateLogTile error in MySQL storage tests #3657

robstradling opened this issue Oct 28, 2024 · 2 comments

Comments

@robstradling
Copy link
Contributor

Here's a snippet from the output of go test -v ./storage/mysql:

=== RUN   TestNodeRoundTrip/store-all-read-all-norevisions
E1028 16:50:59.282101  144538 subtree_cache.go:122] PopulateLogTile(): log repop got: 254 internal nodes, want: 0
=== RUN   TestNodeRoundTrip/store-all-read-all-revisions
E1028 16:50:59.299517  144538 subtree_cache.go:122] PopulateLogTile(): log repop got: 254 internal nodes, want: 0

By trial and error I've found that these PopulateLogTile errors started occurring after this commit. However, that commit's description seems to suggest that it may have only surfaced the issue rather than created it.

@rahulmansharamani14
Copy link

Hi @patflynn,
I’m interested in working on this issue. Could you please let me know if there are any specific guidelines or pointers to keep in mind? Thank you!

@patflynn
Copy link
Contributor

patflynn commented Nov 5, 2024

Hi @rahulmansharamani14 ! I'm really happy to see external interest, and we welcome contributions. Be aware that we're working quite hard on Tessera ATM, so depending on the size of PR and the review challenge you may see some delays.

@AlCutter may have some pointers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants