Xline

Xline copied to clipboard

cargo build --release Compiling console v0.15.2 error: unnecessary parentheses around match arm expression --> curp/src/bg_tasks.rs:191:28 | 191 | Ok(req) => (req), | ^ ^ | note: the lint level is...

bonito-skipjack

To avoid conflicts with KvServer's commands, AuthServer's commands has empty keys field and will always propose with slow path

themanforfree

markcty

help: unitTest error

1

When I tested the method test_kv_put() in xline/tests/kv_test.rs, it have a wrong.

azhsmesos

Currently, configurations like `HEATBEAT_INTERVAL`, `N_EXECUTE_WORKERS`, or `ELECTION_TIMEOUT` are hardcoded in the code. Make it configurable by developing something like a server builder pattern.

markcty

Lease

1

themanforfree

[merged_range](https://github.com/datenlord/merged_range)

themanforfree

themanforfree

this [loop](https://github.com/datenlord/Xline/blob/f816289275a96149d210758e0b8cf6e099228127/xline/src/storage/kvstore.rs#L66-L79) may be blocked by slow requests, and it cannot ensure that the same command will call sync after calling execute

themanforfree

Now we may store several cmds in one log entry. Store them separately. _Originally posted by @rogercloud in https://github.com/datenlord/Xline/pull/85#discussion_r1034393498_

markcty