ins icon indicating copy to clipboard operation
ins copied to clipboard

Update ins_node.proto

Open syeerzy opened this issue 6 years ago • 5 comments

change message ScanRequest's end_key field's type from bytes to string (to be the same as the start_key field)

syeerzy avatar Oct 11 '18 09:10 syeerzy

这个区间应该是前闭后开的,end改成string不一定科学

yvxiang avatar Oct 14 '18 09:10 yvxiang

c++用的少, 没反应过来, 前闭后开跟结束点的类型是string还是bytes 有什么关系呢? 有序,离散的类型都可以前闭后开啊.

syeerzy avatar Oct 15 '18 01:10 syeerzy

比如想读[a, b)之间的所有数据,即:大于等于a,但是小于b,这时区间的end中含'\0',如果使用string就不对了

yvxiang avatar Oct 15 '18 02:10 yvxiang

如果允许 key 是个含有\0的, 那start也有这个问题啊, 如果是
.... aa aaa\0bba aaa\0bbb <- 期望的 start, 实际上传过去的string是 "aaa"

按你说法那就会把前一条给读到了(多读一条). 那如果允许key里中间带着 \0 的话, 应该两个都用 bytes 对吧?

syeerzy avatar Oct 15 '18 03:10 syeerzy

有道理,改掉吧:)

yvxiang avatar Oct 15 '18 03:10 yvxiang