[GitHub] carbondata pull request #911: [CARBONDATA-1050]Changing the datatype of the ...

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

[GitHub] carbondata pull request #911: [CARBONDATA-1050]Changing the datatype of the ...

qiuchenjian-2
GitHub user ravikiran23 opened a pull request:

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

    [CARBONDATA-1050]Changing the datatype of the int,short from long to original data types.

    problem:
    The datatype of the int and short is treated as long datatype.
   
    Solution:
    Need to treat int and short types as their respective datatype. No need to treat it as long.

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

    $ git pull https://github.com/ravikiran23/incubator-carbondata datatype

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

    https://github.com/apache/carbondata/pull/911.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 #911
   
----
commit 28c6d542d206c691aee88fb8e97466a648d616cf
Author: joobisb <[hidden email]>
Date:   2017-05-12T09:00:59Z

    Changing the datatype of the int,short from long to original data types.

----


---
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 #911: [CARBONDATA-1050]Changing the datatype of the int,sho...

qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/911
 
    Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2011/



---
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 pull request #911: [CARBONDATA-1050]Changing the datatype of the ...

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

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


---
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 pull request #911: [CARBONDATA-1050]Changing the datatype of the ...

qiuchenjian-2
In reply to this post by qiuchenjian-2
GitHub user ravikiran23 reopened a pull request:

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

    [CARBONDATA-1050]Changing the datatype of the int,short from long to original data types.

    problem:
    The datatype of the int and short is treated as long datatype.
   
    Solution:
    Need to treat int and short types as their respective datatype. No need to treat it as long.

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

    $ git pull https://github.com/ravikiran23/incubator-carbondata datatype

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

    https://github.com/apache/carbondata/pull/911.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 #911
   
----
commit 28c6d542d206c691aee88fb8e97466a648d616cf
Author: joobisb <[hidden email]>
Date:   2017-05-12T09:00:59Z

    Changing the datatype of the int,short from long to original data types.

----


---
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 #911: [CARBONDATA-1050]Changing the datatype of the int,sho...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:

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


---
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 pull request #911: [CARBONDATA-1050]Changing the datatype of the ...

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

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


---
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.
---