fix error passing
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
This commit is contained in:
parent
09f4a15ec4
commit
8f7d51ed3b
18
util.go
18
util.go
@ -2,6 +2,7 @@ package kgo
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"context"
|
"context"
|
||||||
|
"errors"
|
||||||
"sync"
|
"sync"
|
||||||
|
|
||||||
kgo "github.com/twmb/franz-go/pkg/kgo"
|
kgo "github.com/twmb/franz-go/pkg/kgo"
|
||||||
@ -9,6 +10,8 @@ import (
|
|||||||
"github.com/unistack-org/micro/v3/logger"
|
"github.com/unistack-org/micro/v3/logger"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
var ErrLostMessage = errors.New("message not marked for offsets commit and will be lost in next iteration")
|
||||||
|
|
||||||
var pPool = sync.Pool{
|
var pPool = sync.Pool{
|
||||||
New: func() interface{} {
|
New: func() interface{} {
|
||||||
return &publication{msg: &broker.Message{}}
|
return &publication{msg: &broker.Message{}}
|
||||||
@ -44,7 +47,7 @@ func (s *subscriber) run(ctx context.Context) {
|
|||||||
}
|
}
|
||||||
if len(fetches.Errors()) > 0 {
|
if len(fetches.Errors()) > 0 {
|
||||||
for _, err := range fetches.Errors() {
|
for _, err := range fetches.Errors() {
|
||||||
s.kopts.Logger.Errorf(ctx, "fetch err topic %s partition %d: %v", err.Topic, err.Partition, err.Err)
|
s.kopts.Logger.Fatalf(ctx, "fetch err topic %s partition %d: %v", err.Topic, err.Partition, err.Err)
|
||||||
}
|
}
|
||||||
// TODO: fatal ?
|
// TODO: fatal ?
|
||||||
return
|
return
|
||||||
@ -62,6 +65,9 @@ func (s *subscriber) run(ctx context.Context) {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
select {
|
select {
|
||||||
|
case err := <-w.cherr:
|
||||||
|
s.kopts.Logger.Fatalf(ctx, "handle err: %v", err)
|
||||||
|
return
|
||||||
case w.recs <- p.Records:
|
case w.recs <- p.Records:
|
||||||
case <-w.done:
|
case <-w.done:
|
||||||
}
|
}
|
||||||
@ -159,6 +165,10 @@ func (w *worker) handle() {
|
|||||||
_ = eh(p)
|
_ = eh(p)
|
||||||
if p.ack {
|
if p.ack {
|
||||||
w.reader.MarkCommitRecords(record)
|
w.reader.MarkCommitRecords(record)
|
||||||
|
} else {
|
||||||
|
w.cherr <- ErrLostMessage
|
||||||
|
pPool.Put(p)
|
||||||
|
return
|
||||||
}
|
}
|
||||||
pPool.Put(p)
|
pPool.Put(p)
|
||||||
continue
|
continue
|
||||||
@ -186,9 +196,13 @@ func (w *worker) handle() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
if p.ack {
|
if p.ack {
|
||||||
|
pPool.Put(p)
|
||||||
w.reader.MarkCommitRecords(record)
|
w.reader.MarkCommitRecords(record)
|
||||||
|
} else {
|
||||||
|
pPool.Put(p)
|
||||||
|
w.cherr <- ErrLostMessage
|
||||||
|
return
|
||||||
}
|
}
|
||||||
pPool.Put(p)
|
|
||||||
}
|
}
|
||||||
if paused {
|
if paused {
|
||||||
paused = false
|
paused = false
|
||||||
|
Loading…
Reference in New Issue
Block a user