Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTTP client API all-in-one [3/3] #1398

Merged
merged 7 commits into from
Apr 19, 2016
Merged

HTTP client API all-in-one [3/3] #1398

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
bf62ab8
HTTP client API
Apr 17, 2016
1340a1f
Merge remote branch 'origin/riak_ts-develop' into feature-az+th-http_…
Apr 19, 2016
1912a98
HTTP client API
Apr 17, 2016
fb6ed37
Merge remote branch 'origin/riak_ts-develop' into feature-az+th-http_…
Apr 19, 2016
00296fb
Merge branch 'feature-az+th-http_client_api_allinone' of github.com:b…
Apr 19, 2016
d0e8a82
Revert do_describe to return lists of lists
Apr 19, 2016
94db162
wm_timeseries_listkeys: log a warning on unexpected messages
Apr 19, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions src/riak_kv_qry_worker.erl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ prepare_final_results(#state{
prepare_final_results2(#riak_sel_clause_v1{col_return_types = ColTypes,
col_names = ColNames}, Rows) ->
%% filter out empty records
FinalRows = [list_to_tuple(R) || R <- Rows, R /= [[]]],
FinalRows = [R || R <- Rows, R /= [[]]],
{ColNames, ColTypes, FinalRows}.

%%%===================================================================
Expand All @@ -292,10 +292,8 @@ prepare_final_results2(#riak_sel_clause_v1{col_return_types = ColTypes,

prepare_final_results_test() ->
Rows = [[12, <<"windy">>], [13, <<"windy">>]],
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be tuples then? I'll test and find out...

Copy link
Contributor Author

@hmmr hmmr Apr 19, 2016 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

list_to_tuple now happens in riak_kv_ts_svc, right before the records are delivered to the PB/TTB code and further to the client. This is so in order to avoid converting tuples back to lists in riak_kv_wm_timeseries_query, where lists are required for converting the data to json.

Which is why we still have native records-as-lists here.

RowsAsTuples = [{12, <<"windy">>}, {13, <<"windy">>}],
% IndexedChunks = [{1, Rows}],
?assertEqual(
{[<<"a">>, <<"b">>], [sint64, varchar], RowsAsTuples},
{[<<"a">>, <<"b">>], [sint64, varchar], Rows},
prepare_final_results(
#state{
qry =
Expand Down
5 changes: 3 additions & 2 deletions src/riak_kv_ts_svc.erl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -387,8 +387,9 @@ sub_tscoveragereq(Mod, _DDL, #tscoveragereq{table = Table,
{reply, ts_query_responses() | #rpberrorresp{}, #state{}}.
sub_tsqueryreq(_Mod, DDL = ?DDL{table = Table}, SQL, State) ->
case riak_kv_ts_api:query(SQL, DDL) of
{ok, Data} ->
{reply, make_tsqueryresp(Data), State};
{ok, {ColNames, ColTypes, LdbNativeRows}} ->
Rows = [list_to_tuple(R) || R <- LdbNativeRows],
{reply, make_tsqueryresp({ColNames, ColTypes, Rows}), State};

%% the following timeouts are known and distinguished:
{error, no_type} ->
Expand Down
14 changes: 12 additions & 2 deletions src/riak_kv_web.erl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
%%
%% riak_kv_web: setup Riak's KV HTTP interface
%%
%% Copyright (c) 2007-2010 Basho Technologies, Inc. All Rights Reserved.
%% Copyright (c) 2007-2016 Basho Technologies, Inc. All Rights Reserved.
%%
%% This file is provided to you under the Apache License,
%% Version 2.0 (the "License"); you may not use this file
Expand Down Expand Up @@ -121,7 +121,17 @@ raw_dispatch(Name) ->
{Prefix ++ ["buckets", bucket, "index", field, '*'],
riak_kv_wm_index, Props}

] || {Prefix, Props} <- Props2 ]).
] || {Prefix, Props} <- Props2 ]) ++

lists:flatten(
[
%% Right now we only have version 1. When we get version 2 we have to
%% decide if we want to dispatch to separate resource modules or handle
%% the different versions inside the same resource handler module.
[{["ts", api_version, "tables", table, "list_keys"], riak_kv_wm_timeseries_listkeys, Props},
{["ts", api_version, "tables", table, "keys", '*'], riak_kv_wm_timeseries, Props},
{["ts", api_version, "query"], riak_kv_wm_timeseries_query, Props}
] || {_Prefix, Props} <- Props2]).

is_post(Req) ->
wrq:method(Req) == 'POST'.
Expand Down
Loading