arrow

ARROW-3815 [R]: refine record batch factory

so that `array()` when given an `arrow::Array` is identity

``` r
library(arrow, warn.conflicts = FALSE)

array(array(1:10))
#> arrow::Array
#> [
#>   1,
#>   2,
#>   3,
#>   4,
#>   5,
#>   6,
#>   7,
#>   8,
#>   9,
#>   10
#> ]
```

so that we may supply already made `arrow::Array` in `record_batch()`

```r
batch <- record_batch(x = 1:10, y = arrow::array(1:10))
as_tibble(batch)
#> # A tibble: 10 x 2
#>        x     y
#>    <int> <int>
#>  1     1     1
#>  2     2     2
#>  3     3     3
#>  4     4     4
#>  5     5     5
#>  6     6     6
#>  7     7     7
#>  8     8     8
#>  9     9     9
#> 10    10    10
```

<sup>Created on 2019-06-10 by the [reprex package](https://reprex.tidyverse.org) (v0.3.0.9000)</sup>

Author: Romain Francois <romain@rstudio.com>

Closes #4508 from romainfrancois/ARROW-3815/record_batch and squashes the following commits:

189ab978 <Romain Francois> more idiomatic C++11 range foor loop
9e479c81 <Romain Francois> linting
4fb4f332 <Romain Francois> additional chunked_array() test
f17ae031 <Romain Francois> Array__from_vector() recosgnise arrow::Array
2925a740 <Romain Francois> InferType recognizes arrow::Array