search

agentm / project-m36

Project: M36 Relational Algebra Engine
The Unlicense
876 stars 47 forks source link

Remove `DataFrameTuple`? #318

Closed farzadbekran closed 2 years ago

farzadbekran commented 2 years ago

DataFrameTuple and RelationTuple are exactly the same thing. Shouldn't we use only one of them to make things simpler? The results from DataFrameExpr contain DataFrameTuples and therefore don't compose well with the rest of the code.

agentm commented 2 years ago

This is an intentional choice to make it clear that using DataFrame over Relation is transformative and not isomorphic. Also, the internal RelationTuple structure may change in the future while the DataFrameTuple is logically complete with ordered values.