umartin commented on PR #1041: URL: https://github.com/apache/sedona/pull/1041#issuecomment-1740377533
Just my two cents. I would prefer removing the Sedona Metric class and use the Spark build in LongAccumulator instead. The LongAccumulator has no overhead. There would be no need for a debug option. Less code, less knobs and metrics for everyone. Keeping the Sedona Metric class means there is no way to have metrics for larger jobs, where you would really want them for tuning. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
