fix: scanner.next() won't automatically recover when server side not return ERR_OK

[foreverneverer]

See XiaoMi/pegasus-java-client#156

[codecov]

Codecov Report

Merging #86 (bb86d7e) into master (50cb769) will increase coverage by 2.19%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
+ Coverage   68.75%   70.95%   +2.19%     
==========================================
  Files          19       19              
  Lines        1277     1284       +7     
==========================================
+ Hits          878      911      +33     
+ Misses        355      331      -24     
+ Partials       44       42       -2     

Impacted Files	Coverage Δ
pegasus/scanner.go	87.23% <92.85%> (+11.11%)	⬆️
session/session.go	86.33% <0.00%> (+1.24%)	⬆️
pegasus/table_connector.go	86.45% <0.00%> (+3.22%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 50cb769...bb86d7e. Read the comment docs.

[neverchanje]

LGTM. But you should still need to add a test before merging this PR.

[Smityz]

What's the meaning of batchUnknownError?

[foreverneverer]

What's the meaning of batchUnknownError?

any unhandler error，such as rpc success but rocksdb report un-handle error

[hycdong]

What's the meaning of batchUnknownError?

any unhandler error，such as rpc success but rocksdb report un-handle error

batchUnknownError makes me confused too, I suggest that you can add comments for it.

[foreverneverer]

What's the meaning of batchUnknownError?

any unhandler error，such as rpc success but rocksdb report un-handle error

batchUnknownError makes me confused too, I suggest that you can add comments for it.

@hycdong done