setDT function already grown pretty fat. AFAIK it was meant to be a low overhead function. Making it low overhead now would be challanging, or even impossible, without removing some of its features. Therefore it seems reasonable to introduce new argument, lets call it asis, that will make as few as possible, which is to set:
Such a change would be helpful in case if someone wants to benchmark DT operations but for some reason would like to include time of as.data.table / as.data.frame (time of an extra deep copy). setDT already removes extra deep copy, but they have other stuff, that is unnecessary in a case of just changing class.
setDTfunction already grown pretty fat. AFAIK it was meant to be a low overhead function. Making it low overhead now would be challanging, or even impossible, without removing some of its features. Therefore it seems reasonable to introduce new argument, lets call itasis, that will make as few as possible, which is to set:Such a change would be helpful in case if someone wants to benchmark DT operations but for some reason would like to include time of
as.data.table/as.data.frame(time of an extra deep copy).setDTalready removes extra deep copy, but they have other stuff, that is unnecessary in a case of just changing class.