Fix naming in j=c() under by= queries with lapply() optimization#4883

Merged

MichaelChirico merged 33 commits intoRdatatable:masterfrom

myoung3:named_lapply

Sep 6, 2024

Commits on Jan 31, 2021

added failing tests
Michael Young
committed
successfully fixed 2164.1 but broke others. also, havent fixed named list yet
Michael Young
committed
fixed random broken tests. only the c(a=list(),lapply(.SD,FUN)) situation left
Michael Young
committed
More tests. Some still failing
Michael Young
committed
c(A=list()) shouldn't get a number prefixed
Michael Young
committed
fixed previously failing test. the issue was that I had 2164.1 and 2164.10. oops. also added more tests. all passing now
Michael Young
committed
add to author list
Michael Young
committed

Commits on Feb 1, 2021

added github link to issue
Michael Young
committed

Commits on Feb 3, 2021

code style changes
Michael Young
committed

Commits on Feb 20, 2021

reworked tests. now includes tests that fail only when optimize=0
Michael Young
committed
fix breaking tests related to inconsistent naming of blank columnames
Michael Young
committed
added more passing tests
Michael Young
committed
cleaned up news item
Michael Young
committed
added another passing test ensuring dt[, c(x[1], list(.),] gets a variable name for the column created by x[1]. previously this could be an empty string column name in some circumstances
Michael Young
committed
fix code style
Michael Young
committed

Commits on Sep 1, 2021

Commits on Sep 6, 2024