Implement fit!(Series, DTable) about juliadb.jl HOT 6 CLOSED

@joshday What is your current thinking here. Should this be done in a different way such that this issue can be closed?

from juliadb.jl.

Right now reduce copies a Series argument. It's on my list of things to do to change it to non-copying so things like

s = Series(Hist(50))
reduce(s, mytable, select = :x)
reduce(s, anothertable, select = :x)
plot(s)

will work as expected. It requires a few changes that I haven't had time to dig into the details, but it should be fairly straightforward to do.

from juliadb.jl.

The workaround for now is:

s = Series(Hist(20))
s = reduce(s, mytable; select=:x)
s = reduce(s, anothertable; select=:x)
plot(s)

from juliadb.jl.

Are we going to do this? Or are we just going to use reduce?

from juliadb.jl.

I think we should use reduce. Last time I looked into making reduce non-copying I found an issue in JuliaDB, but everything seemed fine in IndexedTables. Sorting that out is on my to-do list, but low priority right now.

from juliadb.jl.

Okay then. Yeah, I think we copy Stats (in JuliaDB) while merging results from different procs.

from juliadb.jl.