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

Rebase b1.0 #3600

Closed
wants to merge 14 commits into from
Closed

Conversation

webbnh
Copy link
Member

@webbnh webbnh commented Jan 18, 2024

This is an experiment.

dbutenhof and others added 14 commits December 20, 2023 08:12
This combines a few issues: first, I've wanted to filter based on the unpacked
tarball size, but some tarballs are beyond the range of the SQL `INTEGER` type
and cause SQL cast errors -- change the interpretation of the `int` filter and
sort type to `BigInteger`. Also cleans up the logging around retried Sync
transaction errors, only logging warnings when it can't determine that the
error is a PostgreSQL serialization error. (I hope: this is hard to provoke in
casual testing.) Finally, clean up the logging of cached unpacked size by
avoiding two separate logs (without dataset name) on unpack, and adding a log
of the final unpacked size when we compute it.
Sort datasets by uploaded time
* PBENCH-1300
Visualization Page Pagination
Display metadata modal is empty
Overview page displays Public datasets
…s#3595)

* Another tweak to intake metadata problems

Make sure we can't end up with undefined `metadata`.

Record details of `metadata.log` access to `run.controller` without adding
a ton of separate messages.
* Minor logging cleanup

Minimize cache logging: details were useful when cache management first went
in, but are now disruptive during ops review.
@webbnh
Copy link
Member Author

webbnh commented Jan 18, 2024

That was fun. Now I'll try again "for real" in a third PR.

@webbnh webbnh closed this Jan 18, 2024
@webbnh webbnh deleted the rebase_b1.0 branch January 18, 2024 22:18
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.

3 participants