Dear Ivan,
thanks for the confirmation and the proposed patch.
I just wanted to add some notes regarding the relevance of this: base::merge
using by.x=0 or by.y=0 (i.e. matching on row.names) will automatically add a
column Row.names which is I(row.names(x)) to the corresponding input table
(using I() since revision 39026 to avoid conversion of character to factor).
When this column is used for sorting (sort=TRUE by default in merge; should
happen at least if all.x=T or all.y=T), this will result in slower execution.
xtfrm.AsIs is unchanged since its addition in r50992 (likely unrelated to the
former).
So I guess that this just went unnoticed since it will not cause problems on
small data frames.
Best regards
Hilmar
[[alternative HTML version deleted]]
______________________________________________
[email protected] mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel
