couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benoit Chesneau <bchesn...@gmail.com>
Subject Re: git commit: updated refs/heads/1960-paginate-all_dbs to fe4096b
Date Sun, 05 Jan 2014 08:51:39 GMT
There won't be any shasoow naming here but your suggestion is still valid
for code readers so I remaned it to MAX_DBS_LISTED.

Thanks!


On Sun, Jan 5, 2014 at 9:42 AM, Alexander Shorin <kxepal@gmail.com> wrote:

> Wouldn't it create collision with max_dbs_open config setting due to
> shadow naming?
> While I know that these things are different, it may be confusing for
> others.
> --
> ,,,^..^,,,
>
>
> On Sun, Jan 5, 2014 at 12:29 PM,  <benoitc@apache.org> wrote:
> > Updated Branches:
> >   refs/heads/1960-paginate-all_dbs e1ea76789 -> fe4096b0e
> >
> >
> > descriptive name for the maximum number of dbs returned.
> >
> >
> > Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/fe4096b0
> > Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/fe4096b0
> > Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/fe4096b0
> >
> > Branch: refs/heads/1960-paginate-all_dbs
> > Commit: fe4096b0e63d594d3d7f3bcfd6246d42f936de0e
> > Parents: e1ea767
> > Author: Benoit Chesneau <benoitc@apache.org>
> > Authored: Sun Jan 5 09:28:55 2014 +0100
> > Committer: Benoit Chesneau <benoitc@apache.org>
> > Committed: Sun Jan 5 09:28:55 2014 +0100
> >
> > ----------------------------------------------------------------------
> >  src/couchdb/couch_httpd_misc_handlers.erl | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/couchdb/blob/fe4096b0/src/couchdb/couch_httpd_misc_handlers.erl
> > ----------------------------------------------------------------------
> > diff --git a/src/couchdb/couch_httpd_misc_handlers.erl
> b/src/couchdb/couch_httpd_misc_handlers.erl
> > index 0d7fb2b..30ad488 100644
> > --- a/src/couchdb/couch_httpd_misc_handlers.erl
> > +++ b/src/couchdb/couch_httpd_misc_handlers.erl
> > @@ -28,7 +28,7 @@
> >      start_chunked_response/3, send_error/4]).
> >
> >
> > --define(DEFAULT_LIMIT, 16#10000000).
> > +-define(MAX_DBS, 16#10000000).
> >
> >  % httpd global handlers
> >
> > @@ -84,7 +84,7 @@ handle_utils_dir_req(Req, _) ->
> >
> >  handle_all_dbs_req(#httpd{method='GET'}=Req) ->
> >      Limit0 = couch_util:to_integer(couch_httpd:qs_value(Req, "limit",
> > -
>  ?DEFAULT_LIMIT)),
> > +                                                        ?MAX_DBS)),
> >      Skip0 = couch_util:to_integer(couch_httpd:qs_value(Req, "skip",
> -1)),
> >      {ok, {DbNames, _, _}} = couch_server:all_databases(fun
> all_dbs_fun/2,
> >                                                         {[], Skip0,
> Limit0}),
> > @@ -93,7 +93,7 @@ handle_all_dbs_req(Req) ->
> >      send_method_not_allowed(Req, "GET,HEAD").
> >
> >
> > -all_dbs_fun(DbName, {Acc, Skip, 0}) ->
> > +all_dbs_fun(_DbName, {Acc, Skip, 0}) ->
> >      {stop, {Acc, Skip, 0}};
> >  all_dbs_fun(DbName, {Acc, 0, Limit}) ->
> >      {ok, {[DbName | Acc], 0, Limit - 1}};
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message