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

payload: index more tables and quit writing JSON/base64 payloads #1885

Merged
merged 3 commits into from
Apr 2, 2024

Conversation

thoughtpolice
Copy link
Contributor

No description provided.

Signed-off-by: Austin Seipp <[email protected]>
Change-Id: I7a8d1d83453d021c01aca3b8e9c2318d16d6461d
Do not export intermediate payload store function, only export the top-level
`newTransactionDb` and `newPayloadDb` methods. Instead, move these into `where`
blocks so they are scoped appropriately.

This does not change any behavior, because no external clients actually called
any of these. As a bonus, it also means we can remove the now-pointless 'db'
parameter to most functions, since they have it available in their closure.

Signed-off-by: Austin Seipp <[email protected]>
Change-Id: Id1096eb1720a1c309ef808af6610bf986821941a
@thoughtpolice thoughtpolice force-pushed the aseipp/push-sutyrksrttut branch from b483f02 to 2db2973 Compare April 2, 2024 16:24
Signed-off-by: Austin Seipp <[email protected]>
Change-Id: I832611067595ae19b6deeb5e8faf68ec367e7b1e
@thoughtpolice thoughtpolice force-pushed the aseipp/push-sutyrksrttut branch from 2db2973 to 4cd0f1a Compare April 2, 2024 16:29
@thoughtpolice thoughtpolice enabled auto-merge April 2, 2024 16:31
@thoughtpolice thoughtpolice added this pull request to the merge queue Apr 2, 2024
Merged via the queue into master with commit 89807a0 Apr 2, 2024
34 of 39 checks passed
@thoughtpolice thoughtpolice deleted the aseipp/push-sutyrksrttut branch April 2, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants