[GitHub] carbondata pull request #1301: [CARBONDATA-1434] Remove useless class para f...

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #1301: [CARBONDATA-1434] Remove useless class para f...

qiuchenjian-2
GitHub user xuchuanyin opened a pull request:

    https://github.com/apache/carbondata/pull/1301

    [CARBONDATA-1434] Remove useless class para for metastore

    The `RuntimeConfig` has not been used in Carbon*Metastore class, so remove it.

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

    $ git pull https://github.com/xuchuanyin/carbondata useless_class_para

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

    https://github.com/apache/carbondata/pull/1301.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 #1301
   
----
commit 5e886a236a5429a5bef1c8e99eb06fd30828e387
Author: xuchuanyin <[hidden email]>
Date:   2017-08-30T15:14:33Z

    Remove useless class para for metastore

----


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1301: [CARBONDATA-1434] Remove useless class para for meta...

qiuchenjian-2
Github user jackylk commented on the issue:

    https://github.com/apache/carbondata/pull/1301
 
    LGTM


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #1301: [CARBONDATA-1434] Remove useless class para f...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user asfgit closed the pull request at:

    https://github.com/apache/carbondata/pull/1301


---