Merge pull request #5 from queryverse/readonlyarrays

Use ReadOnlyArrays

Author: davidanthoff

Project.toml

@@ -6,6 +6,7 @@ version = "0.1.1-DEV"
 DataValues = "e7dc6d0d-1eca-5fa6-8ad6-5aecde8b7ea5"
 TableShowUtils = "5e66a065-1f0a-5976-b372-e0b8c017ca10"
 TableTraitsUtils = "382cd787-c1b6-5bf2-a167-d5b971a19bda"
+ReadOnlyArrays = "988b38a3-91fc-5605-94a2-ee2116b3bd83"
 
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
@@ -18,3 +19,4 @@ DataValues = "0.4.7"
 TableShowUtils = "0.2.2"
 TableTraitsUtils = "0.4, 1"
 julia = "1"
+ReadOnlyArrays = "0.1.1"

README.md

@@ -35,7 +35,7 @@ To access an individual column by name, use the `.` dot syntax:
 
 ```julia
 julia> dt.Age
-3-element Array{Float64,1}:
+3-element ReadOnlyArrays.ReadOnlyArray{Float64,1,Array{Float64,1}}:
  23.0
  43.0
  56.0

src/QueryTables.jl

@@ -1,6 +1,6 @@
 module QueryTables
 
-import TableShowUtils, TableTraitsUtils
+import TableShowUtils, TableTraitsUtils, ReadOnlyArrays
 
 using DataValues
 
@@ -11,6 +11,7 @@ struct DataTable{T, TCOLS} <: AbstractVector{T}
 end
 
 function fromNT(nt)
+    nt = map(i->i isa ReadOnlyArrays.ReadOnlyArray ? i : ReadOnlyArrays.ReadOnlyArray(i), nt)
     tx = typeof(nt)
     et = NamedTuple{propertynames(nt), Tuple{(eltype(fieldtype(tx, i)) for i in 1:fieldcount(typeof(nt)))...}}
     return DataTable{et, typeof(nt)}(nt)