[GitHub] spark pull request: [SPARK-13164][CORE] Replace deprecated synchro...

2016-02-04 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11059#issuecomment-180052303 sounds like a good rule --- 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

[GitHub] spark pull request: [SPARK-13164][CORE] Replace deprecated synchro...

2016-02-04 Thread tedyu
Github user tedyu commented on the pull request: https://github.com/apache/spark/pull/11059#issuecomment-180046937 Should scalastyle rule be added to ban use of mutable.SynchronizedBuffer ? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-13164][CORE] Replace deprecated synchro...

2016-02-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11059 --- 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

[GitHub] spark pull request: [SPARK-13164][CORE] Replace deprecated synchro...

2016-02-04 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11059#issuecomment-179981487 LGTM merged into master. --- 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

[GitHub] spark pull request: [SPARK-13164][CORE] Replace deprecated synchro...

2016-02-03 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11059#issuecomment-179532488 cc @srowen who expressed interest on the JIRA --- 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