[SQL]Update HiveMetastoreCatalog.scala

[baishuo]

I think it's better to defined hiveQlTable as a val

[AmplabJenkins]

Can one of the admins verify this patch?

[marmbrus]

test this please

[marmbrus]

Please include [SQL] in the PR title for anything that changes Spark SQL.

[SparkQA]

QA tests have started for PR 1569. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17154/consoleFull

[baishuo]

modify the title, add [SQL]

[marmbrus]

I guess it is a def since it is not serializable...

You could make it a @transient lazy val.

BTW you can run the tests locally: sbt/sbt catalyst/test sql/test hive/test

[baishuo]

thank you @marmbrus , I had modify it to "@transient lazy val" and the run "sbt/sbt catalyst/test sql/test hive/test" at master branch , all test passed. if only "val", test can not passed.

the following is the tail of results of test:

[info] - Partition pruning - with filter on string partition key - query test
[info] - Partition pruning - with filter on int partition key - pruning test
[info] - Partition pruning - with filter on int partition key - query test
[info] - Partition pruning - left only 1 partition - pruning test
[info] - Partition pruning - left only 1 partition - query test
[info] - Partition pruning - all partitions pruned - pruning test
[info] - Partition pruning - all partitions pruned - query test
[info] - Partition pruning - pruning with both column key and partition key - pruning test
[info] - Partition pruning - pruning with both column key and partition key - query test
[info] HiveResolutionSuite:
[info] - table.attr
[info] - database.table
[info] - database.table table.attr
[info] - alias.attr
[info] - subquery-alias.attr
[info] - quoted alias.attr
[info] - attr
[info] - alias.star
[info] - case insensitivity with scala reflection
[info] - nested repeated resolution
[info] BigDataBenchmarkSuite:
[info] - No data files found for BigDataBenchmark tests. !!! IGNORED !!!
[info] ScalaTest
[info] Run completed in 2 minutes, 55 seconds.
[info] Total number of tests run: 150
[info] Suites: completed 14, aborted 0
[info] Tests: succeeded 150, failed 0, canceled 0, ignored 7, pending 0
[info] All tests passed.
[info] Passed: Total 150, Failed 0, Errors 0, Passed 150, Ignored 7
[success] Total time: 267 s, completed Jul 25, 2014 10:17:09 AM

[marmbrus]

test this please

[SparkQA]

QA tests have started for PR 1569. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17194/consoleFull

[SparkQA]

QA results for PR 1569:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17194/consoleFull

[marmbrus]

Thanks, I merged this into master.