search

OpenAtomFoundation / pika

Pika is a Redis-Compatible database developed by Qihoo's infrastructure team.
BSD 3-Clause "New" or "Revised" License
5.89k stars 1.19k forks source link

pika parameters are compatible with sentinel #2427

Closed huxiaozhe closed 4 months ago

huxiaozhe commented 8 months ago

Which PikiwiDB functionalities are relevant/related to the feature request?

No response

Description

pika 缺少 master_link_down_since_seconds 指标，含义为 slave 与 master 的链接出现异常之后，slave开始计时。 sentinel选主时就不会选择master_link_down_since_seconds > master宕机时间的 slave 节点，可以更加准确的进行选主。

offset 格式不同 redis ：slave_repl_offset: pika ：db0 binlog_offset=0 0 是否可以抽象出一个 slave_repl_offset

Proposed solution

-

Alternatives considered

-

937988621 commented 6 months ago

您好，请问您说的sentinel是pika自己设计的codis中集成的哨兵还是说redis原生的哨兵呢？我这边在搭建最基础的Master-Slave式的的pika服务时，发现配合redis原生sentinel不能实现故障转移。

Issues-translate-bot commented 6 months ago

Bot detected the issue body's language is not English, translate it automatically.

Hello, is the sentinel you are talking about the sentinel integrated in codis designed by pika itself, or the sentinel native to redis? When I was building the most basic Master-Slave-style pika service, I found that failover could not be achieved with redis's native sentinel.

AlexStocks commented 4 months ago

@baixin01

Issues-translate-bot commented 4 months ago

Bot detected the issue body's language is not English, translate it automatically.

@baixin01

AlexStocks commented 4 months ago

baixin：chengyu 多加了一个冒号导致了此问题。

Issues-translate-bot commented 4 months ago

Bot detected the issue body's language is not English, translate it automatically.

baixin: chengyu added an extra colon, which caused this problem.

chejinge commented 3 months ago

这个问题已经修复解决了

Issues-translate-bot commented 3 months ago

Bot detected the issue body's language is not English, translate it automatically.

This problem has been fixed