added max_results argument to list_tables()

Yep, this is a lot better

On 13 November 2014 06:29, Craig Citro <notifications@github.com> wrote:

> In R/tables.r:
>
> >
> > -  url <- sprintf("projects/%s/datasets/%s/tables", project, dataset)
> > +  url <- sprintf("projects/%s/datasets/%s/tables?maxResults=%d", project, dataset, max_results)
>
> i think we're better off letting the server provide the default if this
> isn't specified.
>
> maybe just leave this line as-is, default max_results to NULL, and
> instead add
>
> if (!is.null(max_results)) {
>   url <- modify_url(url, query = list(maxResults = max_results))
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hadley/bigrquery/pull/36/files#r20274505>.
>