carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mayunSaicmotor <>
Subject [GitHub] incubator-carbondata pull request #646: [CARBONDATA-748] put if clause out o...
Date Sat, 11 Mar 2017 06:50:26 GMT
GitHub user mayunSaicmotor opened a pull request:

    [CARBONDATA-748] put if clause out of the for clause in method setFilterdIndexToBitSet

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
     - [ ] Make sure the PR title is formatted like:
       `[CARBONDATA-<Jira issue #>] Description of pull request`
     - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
           Travis-CI on your fork and ensure the whole test matrix passes).
     - [ ] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [ ] If this contribution is large, please file an Apache
           [Individual Contributor License Agreement](
     - [ ] Testing done
            Please provide details on 
            - Whether new unit test cases have been added or why no new tests are required?
            - What manual testing you have done?
            - Any additional information to help reviewers in testing this change.
     - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella

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

    $ git pull CARBONDATA-748

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

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #646
commit 252649eecee6a7b74eef5a7b7f17d58a363c09ea
Author: mayun <mayun@>
Date:   2017-03-09T05:13:22Z

    use binary search to improve the performance in method

commit c50054fa519cc1004b78941cf88541f7ad838976
Author: mayun <mayun@>
Date:   2017-03-09T07:51:50Z

    add binary range search and add test case

commit 25839b1425986cc95275b5e628e03d3fa8d19103
Author: mayun <mayun@>
Date:   2017-03-09T08:08:21Z

    revert previous change

commit 0644946a8bb9877ccdafd96420b091364d126669
Author: mayun <mayun@>
Date:   2017-03-09T08:38:29Z

    format changed code

commit 516c5541722f12dffe5c709238bbb8a2f64e65dc
Author: mayun <mayun@>
Date:   2017-03-09T09:09:06Z

    change code format to pass check style

commit 141e26425ed7296b661a5382a4fe168e33fb71d1
Author: mayun <mayun@>
Date:   2017-03-09T09:51:22Z

    revert the code to use inverted index

commit 0ca8472786974ea11551843b26ae472d422fc847
Author: mayun <mayun@>
Date:   2017-03-10T15:53:34Z

    add comments and change variables definition style

commit 618923cc8cd71257b34bb576779b90a65951f217
Author: mayun <>
Date:   2017-03-11T04:57:56Z

    put the if clause out of the for clause and added test case

commit 70f50adae2e9436970b5744df07be81df8846457
Author: mayun <>
Date:   2017-03-11T05:15:35Z

    Revert "put the if clause out of the for clause and added test case"
    This reverts commit 618923cc8cd71257b34bb576779b90a65951f217.

commit 7acb4392b4a03fdd0810a12ad9cd5165dc12696b
Author: 马云 <>
Date:   2017-03-11T05:26:00Z

    Merge pull request #3 from apache/master
    merger from master

commit 9aeccb21f58bb5ebb309221da939595ca5da4be3
Author: mayun <>
Date:   2017-03-11T06:43:32Z

    put if clause out of the for clause and add test case


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

View raw message