Skip to content

Commit

Permalink
Merge pull request #320 from akumuli/fix-bulk-protocol
Browse files Browse the repository at this point in the history
Fix bulk-transfer protocol encoding
  • Loading branch information
Lazin authored Sep 10, 2019
2 parents d111d43 + a842313 commit 505ad7b
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,13 @@
Release notes
=============

Version 0.8.71
--------------

BUG FIX

* Fix bulk-transfer encoding

Version 0.8.70
--------------

Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ cmake_minimum_required(VERSION 2.8)

set(APP_VERSION_MAJOR "0")
set(APP_VERSION_MINOR "8")
set(APP_VERSION_PATCH "70")
set(APP_VERSION_PATCH "71")

set(APP_VERSION "${APP_VERSION_MAJOR}.${APP_VERSION_MINOR}.${APP_VERSION_PATCH}")
add_definitions(-DAKU_VERSION="${APP_VERSION}")
Expand Down
2 changes: 1 addition & 1 deletion akumulid/protocolparser.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ bool RESPProtocolParser::parse_values(RESPStream& stream,
int arrsize;
bool success;
auto parse_int_value = [&](int at) {
std::tie(success, values[0]) = stream.read_int();
std::tie(success, values[at]) = stream.read_int();
if (!success) {
return false;
}
Expand Down

0 comments on commit 505ad7b

Please sign in to comment.