GitHub user DoingDone9 opened a pull request:
https://github.com/apache/spark/pull/3219
[SPARK-4353][SQL] Delete the val that never used in Catalog
"dbName" in Catalog never used, like that
{
val (dbName, tblName) = processDatabaseAndTableName(databaseName,
tableName)
tables -= tblName
}
i think it should be deleted,it should be "val tblName =
processDatabaseAndTableName(databaseName, tableName)._2"
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DoingDone9/spark catalog_issue
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/3219.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3219
----
commit 255107175bf2e51e7f10b77210c3fc93e1b20079
Author: DoingDone9 <[email protected]>
Date: 2014-11-06T01:45:39Z
Merge pull request #1 from apache/master
merge lastest spark
commit 1b392f68830dc22c6c382036419725569aad4b38
Author: DoingDone9 <[email protected]>
Date: 2014-11-11T08:34:41Z
Merge pull request #2 from apache/master
merge lastest spark
commit 73dd949628672bbd07512743f1a66e46ff2e7034
Author: DoingDone9 <[email protected]>
Date: 2014-11-12T02:15:46Z
Merge pull request #3 from apache/master
merge lastest spark
commit 757d45a57d2debfb0dbd1f28fe73ab4061b01870
Author: DoingDone9 <[email protected]>
Date: 2014-11-12T02:27:50Z
Delete the "dbName" that never used
commit 1087237ab20ca02f2c1e8ec2b8255b2a0ec85a07
Author: DoingDone9 <[email protected]>
Date: 2014-11-12T02:34:57Z
Change "db" to "databaseName"
it should be consistent
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
