Github user chiwanpark commented on the pull request:
https://github.com/apache/flink/pull/861#issuecomment-131425060
Hi @sachingoel0101, Thanks for your contribution. I reviewed this PR and
commented the source code.
There are some problems which aren't commented. In documentation, there are
many lines including `<a>`, `<ul>`, `<li>`, '<br>', '<strong>' and '<i>' tag.
These can be replaced with markdown syntax. And some codes
(`MLUtils.createHistogram`, `ColumnStatistics`) are not formatted.
I didn't finish reviewing process of building histogram. After reading full
source code and the given paper, I'll add comment about the process.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
