You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by DoingDone9 <gi...@git.apache.org> on 2015/04/17 11:20:21 UTC

[GitHub] spark pull request: [SPARK-6976][SQL] "drop table if exists src" p...

GitHub user DoingDone9 opened a pull request:

    https://github.com/apache/spark/pull/5553

    [SPARK-6976][SQL] "drop table if exists src" print ERROR info that should not be printed when "src" not exists.

    If table "src" not exists and run sql "drop table if exists src", then some ERROR info will be printed, like that
    ```
    15/04/17 17:09:53 ERROR Hive: NoSuchObjectException(message:default.src table not found)
    at org.apache.hadoop.hive.metastore.HiveMetaStore$HMSHandler.get_table(HiveMetaStore.java:1560)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at org.apache.hadoop.hive.metastore.RetryingHMSHandler.invoke(RetryingHMSHandler.java:105)
    at $Proxy10.get_table(Unknown Source)
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DoingDone9/spark drop_table_exists

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/5553.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 #5553
    
----
commit c3f046f8de7c418d4aa7e74afea9968a8baf9231
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-02T02:11:18Z

    Merge pull request #1 from apache/master
    
    merge lastest spark

commit cb1852d14f62adbd194b1edda4ec639ba942a8ba
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-05T07:05:10Z

    Merge pull request #2 from apache/master
    
    merge lastest spark

commit c87e8b6d8cb433376a7d14778915006c31f6c01c
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-10T07:46:12Z

    Merge pull request #3 from apache/master
    
    merge lastest spark

commit 161cae3a29951d793ce721f9904888bd9529de72
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-12T06:46:28Z

    Merge pull request #4 from apache/master
    
    merge lastest spark

commit 98b134f39ca57f11a5b761c7b9e5f8a7477bd069
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-19T09:00:07Z

    Merge pull request #5 from apache/master
    
    merge lastest spark

commit d00303b7af9436b9bd6d6d27d411a5c8a2e2294d
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-24T08:43:44Z

    Merge pull request #6 from apache/master
    
    merge lastest spark

commit 802261c043f56bd5ebe9e46b15e33cdc7c212176
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-26T02:21:24Z

    Merge pull request #7 from apache/master
    
    merge lastest spark

commit 34b1a9a8a30f689b41fd52b8a10c08666c2ff2b5
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-08T07:55:24Z

    Merge pull request #8 from apache/master
    
    merge lastest spark

commit f61210c03f693a266969e06c52c23ccd1bfe3e1b
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:10:48Z

    Merge pull request #9 from apache/master
    
    merge lastest spark

commit c783d02f5fdc44b894d4e8010d3c26c4cde7850c
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:13:44Z

    Update HiveMetastoreCatalog.scala

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-6976][SQL] "drop table if exists src" p...

Posted by DoingDone9 <gi...@git.apache.org>.
GitHub user DoingDone9 reopened a pull request:

    https://github.com/apache/spark/pull/5553

    [SPARK-6976][SQL] "drop table if exists src" print ERROR info that should not be printed when "src" not exists.

    If table "src" not exists and run sql "drop table if exists src", then some ERROR info will be printed, like that
    ```
    15/04/17 17:09:53 ERROR Hive: NoSuchObjectException(message:default.src table not found)
    at org.apache.hadoop.hive.metastore.HiveMetaStore$HMSHandler.get_table(HiveMetaStore.java:1560)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:601)
    at org.apache.hadoop.hive.metastore.RetryingHMSHandler.invoke(RetryingHMSHandler.java:105)
    at $Proxy10.get_table(Unknown Source)
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DoingDone9/spark drop_table_exists

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/5553.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 #5553
    
----
commit c3f046f8de7c418d4aa7e74afea9968a8baf9231
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-02T02:11:18Z

    Merge pull request #1 from apache/master
    
    merge lastest spark

commit cb1852d14f62adbd194b1edda4ec639ba942a8ba
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-05T07:05:10Z

    Merge pull request #2 from apache/master
    
    merge lastest spark

commit c87e8b6d8cb433376a7d14778915006c31f6c01c
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-10T07:46:12Z

    Merge pull request #3 from apache/master
    
    merge lastest spark

commit 161cae3a29951d793ce721f9904888bd9529de72
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-12T06:46:28Z

    Merge pull request #4 from apache/master
    
    merge lastest spark

commit 98b134f39ca57f11a5b761c7b9e5f8a7477bd069
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-19T09:00:07Z

    Merge pull request #5 from apache/master
    
    merge lastest spark

commit d00303b7af9436b9bd6d6d27d411a5c8a2e2294d
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-24T08:43:44Z

    Merge pull request #6 from apache/master
    
    merge lastest spark

commit 802261c043f56bd5ebe9e46b15e33cdc7c212176
Author: DoingDone9 <79...@qq.com>
Date:   2015-03-26T02:21:24Z

    Merge pull request #7 from apache/master
    
    merge lastest spark

commit 34b1a9a8a30f689b41fd52b8a10c08666c2ff2b5
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-08T07:55:24Z

    Merge pull request #8 from apache/master
    
    merge lastest spark

commit f61210c03f693a266969e06c52c23ccd1bfe3e1b
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:10:48Z

    Merge pull request #9 from apache/master
    
    merge lastest spark

commit c783d02f5fdc44b894d4e8010d3c26c4cde7850c
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:13:44Z

    Update HiveMetastoreCatalog.scala

commit bf38279845efc7ac0adf0764562f4792f60554dd
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:22:26Z

    Update HiveMetastoreCatalog.scala

commit 270fb20d7b56bc5afd7fbe440c3fe96a6dc509a4
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-17T09:36:03Z

    Update HiveMetastoreCatalog.scala

commit d23c0e0dc875d993b614f01aa859dd9f0ebc717f
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-28T09:24:21Z

    Update HiveMetastoreCatalog.scala

commit 47747f9aefa05da9e9691a07ba30c652f9b824fb
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-28T09:26:09Z

    Update HiveQl.scala

commit 45a287ee9afb30ce4a8e89ec5598890216d93809
Author: Zhongshuai Pei <79...@qq.com>
Date:   2015-04-28T09:27:47Z

    Update commands.scala

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-6976][SQL] "drop table if exists src" p...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5553#issuecomment-93950877
  
    Can one of the admins verify this patch?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-6976][SQL] "drop table if exists src" p...

Posted by DoingDone9 <gi...@git.apache.org>.
Github user DoingDone9 closed the pull request at:

    https://github.com/apache/spark/pull/5553


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-6976][SQL] "drop table if exists src" p...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5553#issuecomment-96988713
  
    Can one of the admins verify this patch?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org