-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-9928][SQL] Removal of LogicalLocalTable #9717
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…icalLocalTable merge.
…icalLocalTable
…icalLocalTable
Test build #45933 has finished for PR 9717 at commit
|
The failure of this test case is not related to the code changes. It sounds like the other PR also hit the same issue. For example, in #9642, the same failure: |
Anybody can help me trigger the retest, please? |
@srowen Could you review the changes? Thanks! |
Test build #45935 has finished for PR 9717 at commit
|
Another case failed due to the same reasons. This is not related to the code changes made in this PR.
Timing issues? Or introduced by a recent merge? |
retest this please. |
the tests are quite flaky now, we are fixing it. |
Test build #45969 has finished for PR 9717 at commit
|
LGTM cc @yhuai |
Merging to master and branch 1.6. |
LogicalLocalTable in ExistingRDD.scala is replaced by localRelation in LocalRelation.scala? Do you know any reason why we still keep this class? Author: gatorsmile <gatorsmile@gmail.com> Closes #9717 from gatorsmile/LogicalLocalTable. (cherry picked from commit b58765c) Signed-off-by: Yin Huai <yhuai@databricks.com>
Thanks! |
LogicalLocalTable in ExistingRDD.scala is replaced by localRelation in LocalRelation.scala?
Do you know any reason why we still keep this class?