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.

Sign up for GitHub

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

Jump to bottom

Fixed 64-bit alignment panic on 32-bit platforms #290

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed 64-bit alignment panic on 32-bit platforms #290

wants to merge 1 commit into from

Conversation

shemanaev
Copy link

It's the same story as per #148, but for v16.

panic: unaligned 64-bit atomic operation

goroutine 1 [running]:
internal/runtime/atomic.panicUnaligned()
/root/go/pkg/mod/golang.org/[email protected]/src/internal/runtime/atomic/unaligned.go:8 +0x2d
internal/runtime/atomic.Xadd64(0xa0226e4, 0x0)
/root/go/pkg/mod/golang.org/[email protected]/src/internal/runtime/atomic/atomic_386.s:125 +0x11
github.com/blevesearch/zapx/v16.(*invertedIndexOpaque).incrementBytesWritten(...)
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/section_inverted_text_index.go:401
github.com/blevesearch/zapx/v16.(*invertedIndexOpaque).writeDicts(0xa022608, 0xfd7efa8)
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/section_inverted_text_index.go:529 +0xe74
github.com/blevesearch/zapx/v16.(*invertedTextIndexSection).Persist(0x9424880, 0xfe720c0, 0xfd7efa8)
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/section_inverted_text_index.go:54 +0x3d
github.com/blevesearch/zapx/v16.(*interim).convert(0xfe5fc00)
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/new.go:220 +0x6d9
github.com/blevesearch/zapx/v16.(*ZapPlugin).newWithChunkMode(0x9424880, {0xf774008, 0x3e9, 0x3e9}, 0x402)
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/new.go:69 +0x194
github.com/blevesearch/zapx/v16.(*ZapPlugin).New(0x9424880, {0xf774008, 0x3e9, 0x3e9})
/root/go/pkg/mod/github.com/blevesearch/zapx/[email protected]/new.go:45 +0x43
github.com/blevesearch/bleve/v2/index/scorch.(*Scorch).Batch(0x9fde008, 0xf6ffa90)
/root/go/pkg/mod/github.com/blevesearch/bleve/[email protected]/index/scorch/scorch.go:445 +0x63a
github.com/blevesearch/bleve/v2.(*indexImpl).Batch(0x9c98500, 0xf6bf680)
/root/go/pkg/mod/github.com/blevesearch/bleve/[email protected]/index_impl.go:316 +0x90

shemanaev added a commit to shemanaev/inpxer that referenced this pull request Jan 7, 2025
@shemanaev
Add support for 32 bit cpus
5ed82ac 
badger doesn't play nice with 386, so [pebble](https://github.com/cockroachdb/pebble) was used as replacement.
Also waiting for blevesearch/zapx#290 to be merger
@shemanaev shemanaev mentioned this pull request Jan 7, 2025
Open
@abhinavdangeti
Copy link
Member

@Thejas-bhat would you review this one please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abhinavdangeti abhinavdangeti Awaiting requested review from abhinavdangeti

@Thejas-bhat Thejas-bhat Awaiting requested review from Thejas-bhat

@metonymic-smokey metonymic-smokey Awaiting requested review from metonymic-smokey

@CascadingRadium CascadingRadium Awaiting requested review from CascadingRadium

@Likith101 Likith101 Awaiting requested review from Likith101

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shemanaev @abhinavdangeti