{"push_id":1412116904,"size":4,"distinct_size":4,"ref":"refs/heads/master","head":"ded5fefb6f5c0a97bf3d7fa1c0494dc434b6ee40","before":"2a40de408b5eb47edba92f9fe92a42ed1e78bf98","commits":[{"sha":"db9fb9baacbf8640dd37a507b7450db727c7e6ea","author":{"name":"Sean Owen","email":"4c59f357315334980589bde1049040bf6de6532c@cloudera.com"},"message":"[SPARK-18448][CORE] SparkSession should implement java.lang.AutoCloseable like JavaSparkContext\n\n## What changes were proposed in this pull request?\n\nJust adds `close()` + `Closeable` as a synonym for `stop()`. This makes it usable in Java in try-with-resources, as suggested by ash211  (`Closeable` extends `AutoCloseable` BTW)\n\n## How was this patch tested?\n\nExisting tests\n\nAuthor: Sean Owen <sowen@cloudera.com>\n\nCloses #15932 from srowen/SPARK-18448.","distinct":true,"url":"https://api.github.com/repos/ueshin/apache-spark/commits/db9fb9baacbf8640dd37a507b7450db727c7e6ea"},{"sha":"d5b1d5fc80153571c308130833d0c0774de62c92","author":{"name":"hyukjinkwon","email":"4c114bf3676ebcdb1616a11ca478d9181f4025da@gmail.com"},"message":"[SPARK-18445][BUILD][DOCS] Fix the markdown for `Note:`/`NOTE:`/`Note that`/`'''Note:'''` across Scala/Java API documentation\n\n## What changes were proposed in this pull request?\n\nIt seems in Scala/Java,\n\n- `Note:`\n- `NOTE:`\n- `Note that`\n- `'''Note:'''`\n- `note`\n\nThis PR proposes to fix those to `note` to be consistent.\n\n**Before**\n\n- Scala\n  ![2016-11-17 6 16 39](https://cloud.githubusercontent.com/assets/6477701/20383180/1a7aed8c-acf2-11e6-9611-5eaf6d52c2e0.png)\n\n- Java\n  ![2016-11-17 6 14 41](https://cloud.githubusercontent.com/assets/6477701/20383096/c8ffc680-acf1-11e6-914a-33460bf1401d.png)\n\n**After**\n\n- Scala\n  ![2016-11-17 6 16 44](https://cloud.githubusercontent.com/assets/6477701/20383167/09940490-acf2-11e6-937a-0d5e1dc2cadf.png)\n\n- Java\n  ![2016-11-17 6 13 39](https://cloud.githubusercontent.com/assets/6477701/20383132/e7c2a57e-acf1-11e6-9c47-b849674d4d88.png)\n\n## How was this patch tested?\n\nThe notes were found via\n\n```bash\ngrep -r \"NOTE: \" . | \\ # Note:|NOTE:|Note that|'''Note:'''\ngrep -v \"// NOTE: \" | \\  # starting with // does not appear in API documentation.\ngrep -E '.scala|.java' | \\ # java/scala files\ngrep -v Suite | \\ # exclude tests\ngrep -v Test | \\ # exclude tests\ngrep -e 'org.apache.spark.api.java' \\ # packages appear in API documenation\n-e 'org.apache.spark.api.java.function' \\ # note that this is a regular expression. So actual matches were mostly `org/apache/spark/api/java/functions ...`\n-e 'org.apache.spark.api.r' \\\n...\n```\n\n```bash\ngrep -r \"Note that \" . | \\ # Note:|NOTE:|Note that|'''Note:'''\ngrep -v \"// Note that \" | \\  # starting with // does not appear in API documentation.\ngrep -E '.scala|.java' | \\ # java/scala files\ngrep -v Suite | \\ # exclude tests\ngrep -v Test | \\ # exclude tests\ngrep -e 'org.apache.spark.api.java' \\ # packages appear in API documenation\n-e 'org.apache.spark.api.java.function' \\\n-e 'org.apache.spark.api.r' \\\n...\n```\n\n```bash\ngrep -r \"Note: \" . | \\ # Note:|NOTE:|Note that|'''Note:'''\ngrep -v \"// Note: \" | \\  # starting with // does not appear in API documentation.\ngrep -E '.scala|.java' | \\ # java/scala files\ngrep -v Suite | \\ # exclude tests\ngrep -v Test | \\ # exclude tests\ngrep -e 'org.apache.spark.api.java' \\ # packages appear in API documenation\n-e 'org.apache.spark.api.java.function' \\\n-e 'org.apache.spark.api.r' \\\n...\n```\n\n```bash\ngrep -r \"'''Note:'''\" . | \\ # Note:|NOTE:|Note that|'''Note:'''\ngrep -v \"// '''Note:''' \" | \\  # starting with // does not appear in API documentation.\ngrep -E '.scala|.java' | \\ # java/scala files\ngrep -v Suite | \\ # exclude tests\ngrep -v Test | \\ # exclude tests\ngrep -e 'org.apache.spark.api.java' \\ # packages appear in API documenation\n-e 'org.apache.spark.api.java.function' \\\n-e 'org.apache.spark.api.r' \\\n...\n```\n\nAnd then fixed one by one comparing with API documentation/access modifiers.\n\nAfter that, manually tested via `jekyll build`.\n\nAuthor: hyukjinkwon <gurwls223@gmail.com>\n\nCloses #15889 from HyukjinKwon/SPARK-18437.","distinct":true,"url":"https://api.github.com/repos/ueshin/apache-spark/commits/d5b1d5fc80153571c308130833d0c0774de62c92"},{"sha":"8b1e1088eb274fb15260cd5d6d9508d42837a4d6","author":{"name":"Sean Owen","email":"4c59f357315334980589bde1049040bf6de6532c@cloudera.com"},"message":"[SPARK-18353][CORE] spark.rpc.askTimeout defalut value is not 120s\n\n## What changes were proposed in this pull request?\n\nAvoid hard-coding spark.rpc.askTimeout to non-default in Client; fix doc about spark.rpc.askTimeout default\n\n## How was this patch tested?\n\nExisting tests\n\nAuthor: Sean Owen <sowen@cloudera.com>\n\nCloses #15833 from srowen/SPARK-18353.","distinct":true,"url":"https://api.github.com/repos/ueshin/apache-spark/commits/8b1e1088eb274fb15260cd5d6d9508d42837a4d6"},{"sha":"ded5fefb6f5c0a97bf3d7fa1c0494dc434b6ee40","author":{"name":"Sean Owen","email":"4c59f357315334980589bde1049040bf6de6532c@cloudera.com"},"message":"[SPARK-18448][CORE] Fix @since 2.1.0 on new SparkSession.close() method\n\n## What changes were proposed in this pull request?\n\nFix since 2.1.0 on new SparkSession.close() method. I goofed in https://github.com/apache/spark/pull/15932 because it was back-ported to 2.1 instead of just master as originally planned.\n\nAuthor: Sean Owen <sowen@cloudera.com>\n\nCloses #15938 from srowen/SPARK-18448.2.","distinct":true,"url":"https://api.github.com/repos/ueshin/apache-spark/commits/ded5fefb6f5c0a97bf3d7fa1c0494dc434b6ee40"}]}
{"actor":{"display_login":"ueshin"}}