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

在Consumer 里面有个方法叫startProducers,感觉有点怪 #12

Open
zhaoweiguo opened this issue Jun 30, 2021 · 4 comments
Open

Comments

@zhaoweiguo
Copy link

https://github.com/tal-tech/go-queue/blob/3c97bd42eb20ac0356338e10f3fdf47d3746506b/kq/queue.go#L158

上面这个函数,我理解的是这是 Consumer 从Kafka取数据的啊,为啥起名 startProducers。是我哪理解有问题吗?

@kevwan
Copy link
Contributor

kevwan commented Jul 1, 2021

嗯嗯,这个名字确实容易跟kafka的冲突,在queue内部,拿的叫producer, 消费的叫consumer

@zhaoweiguo
Copy link
Author

嗯嗯,这个名字确实容易跟 kafka 的冲突,在 queue 内部,拿的叫 producer, 消费的叫 consumer

嗯,看的时候是很奇怪。如果想适配其他 queue,建议至少在这儿加个注释说明一下

@kevwan
Copy link
Contributor

kevwan commented Jul 1, 2021

嗯嗯,这个名字确实容易跟 kafka 的冲突,在 queue 内部,拿的叫 producer, 消费的叫 consumer

嗯,看的时候是很奇怪。如果想适配其他 queue,建议至少在这儿加个注释说明一下

我们也确实实现了好几个不同的queue

@zhaoweiguo
Copy link
Author

zhaoweiguo commented Jul 12, 2021

嗯嗯,这个名字确实容易跟 kafka 的冲突,在 queue 内部,拿的叫 producer, 消费的叫 consumer

嗯,看的时候是很奇怪。如果想适配其他 queue,建议至少在这儿加个注释说明一下

我们也确实实现了好几个不同的 queue

我刚细看了下和其他 queue 没有耦合,因为另一个dq并没有需要内部先 producer再 consumer

func (q *kafkaQueue) startConsumers() {
	for i := 0; i < q.c.Processors; i++ {
func (q *kafkaQueue) startProducers() {
	for i := 0; i < q.c.Consumers; i++ {

这名字实在太奇怪了,看的时候总得在脑子转个几圈。

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

No branches or pull requests

2 participants