Refactor Map with row function syntax #291

[davibarreira]

Addressing #290.

Enables row functions such as:

Map((row->row.a + row.b) => :NewColumnName)
Map(row->sum(row) => :NewColumnName)

This is just an exploration, so I did not fix the test for show.

[juliohm]

Thank you @davibarreira for performing this experiment! I believe that this syntax is somewhat more limiting than the one we currently have. Does it perform much better if we rely on the Tables.rows directly?

[davibarreira]

I haven't checked. I'll do some benchmarks to see if there is improvement.

[juliohm]

In any case I think the syntax is limited compared to what we can do today. Should we just close it? The performance benefit may not pay off in this case. Em qui., 19 de set. de 2024, 13:27, Davi Sales Barreira < ***@***.***> escreveu:

…

I haven't checked. I'll do some benchmarks to see if there is improvement. — Reply to this email directly, view it on GitHub <#292 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZQW3JZF7PUH2Y5HY3FMA3ZXL3Q3AVCNFSM6AAAAABON5RALGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRRGQ3TONBWGI> . You are receiving this because you commented.Message ID: ***@***.***>