Skip to content

Sync fork #13

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

Open
wants to merge 15 commits into
base: copy-master
Choose a base branch
from
Open

Sync fork #13

wants to merge 15 commits into from

Conversation

kev1n80
Copy link
Owner

@kev1n80 kev1n80 commented Sep 15, 2023

Start the the sync process by merging master branch with this branch that contains nsqio:master updated code.

Fazal Majid and others added 15 commits July 2, 2021 11:17
…a queue was not synced on previous shutdown
avoid corruption when metadata is inconsistent (writePos vs write file size)
Inaccurate depth count after readError on "current" file
CI: switch from Travis-CI to GitHub Actions
debian-based images have gcc (needed for -race) and git
(possible with alpine image but would need more install steps)

go test -race is not supported on 386

Also, go test without -v, but list tests first:
go-diskqueue tests produce a lot of distracting error logging output,
and if a test fails, a pretty good amount of logging context is provided.
switch Travic-CI badge to a GitHub Actions badge

update GoDoc badge to pkg.go.dev "Go Reference" badge
(it was already being redirecting to the new site)
GitHub Actions fixups, README badges update
rather than switching *after* maxBytesPerFile is reached

fixes nsqio#30
switch to next file before maxBytesPerFile is reached
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.

4 participants