fmcquillan99 commented on issue #459: DL: Add support for asymmetric segment distribution to preprocessor URL: https://github.com/apache/madlib/pull/459#issuecomment-558418975 (6) I think it is good that we now have 2 output fields in the summary table: ``` distribution_rules & __internal_gpu_config__ ``` but I would suggest the output should be: ``` if 'distribution_rules' = 'all_segments' distribution_rules | all_segments __internal_gpu_config__ | all_segments if 'distribution_rules' = 'gpu_segments' distribution_rules | {2,3,4,5,6,...} i.e., list out the dbids __internal_gpu_config__ | {0,1,2,3,4...} i.e., list out the content ids if 'distribution_rules' = name of a file containing the list of dbid to use distribution_rules | {2,3,4,5,6,...} i.e., list out the dbids __internal_gpu_config__ | {0,1,2,3,4...} i.e., list out the content ids
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
