Github user danielblazevski commented on the pull request: https://github.com/apache/flink/pull/1220#issuecomment-163438805 @tillrohrmann and @chiwanpark made changes based on your comments. Also note that I recenly got a new laptop, and I decided to install IntelliJ and download the most up-to-date version of Flink's source code and added the 4 relevant files (KNN.scala, QuadTree.scala and the 2 associated test suites) and work on that. Did not realize that this would affect my PR, now the PR shows that 301 files have been changed :-(
--- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---