{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/21531","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/21531/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/21531/comments","events_url":"https://api.github.com/repos/apache/spark/issues/21531/events","html_url":"https://github.com/apache/spark/pull/21531","id":331331501,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk0MDg5MDk0","number":21531,"title":"[SPARK-24521][SQL][TEST] Fix ineffective test in CachedTableSuite","user":{"login":"icexelloss","id":793516,"node_id":"MDQ6VXNlcjc5MzUxNg==","avatar_url":"https://avatars0.githubusercontent.com/u/793516?v=4","gravatar_id":"","url":"https://api.github.com/users/icexelloss","html_url":"https://github.com/icexelloss","followers_url":"https://api.github.com/users/icexelloss/followers","following_url":"https://api.github.com/users/icexelloss/following{/other_user}","gists_url":"https://api.github.com/users/icexelloss/gists{/gist_id}","starred_url":"https://api.github.com/users/icexelloss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icexelloss/subscriptions","organizations_url":"https://api.github.com/users/icexelloss/orgs","repos_url":"https://api.github.com/users/icexelloss/repos","events_url":"https://api.github.com/users/icexelloss/events{/privacy}","received_events_url":"https://api.github.com/users/icexelloss/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2018-06-11T20:08:24Z","updated_at":"2018-06-14T01:47:51Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/21531","html_url":"https://github.com/apache/spark/pull/21531","diff_url":"https://github.com/apache/spark/pull/21531.diff","patch_url":"https://github.com/apache/spark/pull/21531.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\ntest(\"withColumn doesn't invalidate cached dataframe\") in CachedTableSuite doesn't not work because:\r\n\r\nThe UDF is executed and test count incremented when \"df.cache()\" is called and the subsequent \"df.collect()\" has no effect on the test result. \r\n\r\nThis PR fixed this test and add another test for caching UDF.\r\n\r\n## How was this patch tested?\r\n\r\nAdd new tests.\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/397140849","html_url":"https://github.com/apache/spark/pull/21531#issuecomment-397140849","issue_url":"https://api.github.com/repos/apache/spark/issues/21531","id":397140849,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NzE0MDg0OQ==","user":{"login":"SparkQA","id":8108735,"node_id":"MDQ6VXNlcjgxMDg3MzU=","avatar_url":"https://avatars1.githubusercontent.com/u/8108735?v=4","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2018-06-14T01:47:51Z","updated_at":"2018-06-14T01:47:51Z","author_association":"NONE","body":"**[Test build #91800 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91800/testReport)** for PR 21531 at commit [`c9db68d`](https://github.com/apache/spark/commit/c9db68d87a6f34f1849aadc3eaf58ed183cc2419).\n * This patch **fails Spark unit tests**.\n * This patch merges cleanly.\n * This patch adds no public classes."}}
{"actor":{"display_login":"SparkQA"}}