search

gisaia / ARLAS-proc

Workaround about data ingestion with computing frameworks
Apache License 2.0
4 stars 0 forks source link

Prepare FragmentSummary for CourseExtractor #163

Closed laurent-thiebaud-gisaia closed 4 years ago

laurent-thiebaud-gisaia commented 4 years ago

Update some outdated comments. Rename getAggregations to getProcessColumnsForResultRows because there aren't only aggregations. Manage null values by creating the new result row. Remove afterAggregations method, finally useless.

laurent-thiebaud-gisaia commented 4 years ago

@sfalquier I updated the comments and renamed getProcessColumnsForResultRows, is that better for you?