Open bakoe opened 4 years ago
[...] Ihr würde allerdings damit einen weiteren Vektor als Parameter übergeben, was langsam durch das stetige Wachstum nicht gerade schöner wird. Daher meine Bitte: könnt ihr mal bitte gucken, ob es mit vertretbaren Aufwand möglich ist, die drei Vektoren (column_data_distributions, column_data_types, column_data_types) bzw. dann ja bald vier Vektoren durch einen Vektor mit einem einfachen Struct zu ersetzen, der die gleichen Werte hält, dies jedoch per Spalte.
Also z.B. ein Vektor von “SyntheticColumnDefinition” als Struct mit einem optionalen String als Namen, einer Verteilung (Default Uniform) etc. Sollte weitaus sauberer sein. Wenn dies jedoch einen großen Aufwand nach sich zieht (bitte sprecht das mit Thomas ab), dann nehmt die unschönere Variante mit den vier Vektoren. [...] from @Bouncner's email from December 16th
(Will be resolved by hyrise/hyrise/pull/1987)
Based on the feedback from @Bouncner, let's refactor our the SyntheticTableGenerator as follows:
std::optional<std::vector<float>>
instead of astd::optional<float>
(i. e. allow setting the null_ratio per column instead of for all columns at once)SyntheticColumnDefinition
struct used within astd::vector< SyntheticColumnDefinition>
parameter that replaces the three currently (then four) existingstd::vector<>
(orstd::optional<std::vector<>>
) parameters used in the method signature ofSyntheticTableGenerator::generate_table
Notice: On this occasion, create a new branch created from the upstream master to only contain our commits regarding the SyntheticTableGenerator, maybe in a squashed form. Then, create a PR to the upstream to include those changes as soon as possible in Hyrise.