[GitHub] incubator-quickstep issue #279: Applied WorkOrderSelectionPolicy.

2017-08-18 Thread jianqiao
Github user jianqiao commented on the issue:

https://github.com/apache/incubator-quickstep/pull/279
  
I think the PR looks good and the code structure is convenient for further 
adjustment. So merge it now so that @zuyu can continue the work on 
partition-aware scheduling.


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


[GitHub] incubator-quickstep issue #279: Applied WorkOrderSelectionPolicy.

2017-08-14 Thread pateljm
Github user pateljm commented on the issue:

https://github.com/apache/incubator-quickstep/pull/279
  
Nice job @zuyu! @hbdeshmukh can we close this PR, if it looks good to you?


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


[GitHub] incubator-quickstep issue #279: Applied WorkOrderSelectionPolicy.

2017-08-11 Thread zuyu
Github user zuyu commented on the issue:

https://github.com/apache/incubator-quickstep/pull/279
  
Hi @hbdeshmukh, you may want to take a look at this PR as well.

|  **TPC-H** | **master** | **this PR** |
|  -- | --: | --: |
|  01 | 5,238 | 5,040 |
|  02 | 358 | 305 |
|  03 | 1,050 | 1,180 |
|  04 | 2,430 | 2,308 |
|  05 | 5,801 | 2,660 |
|  06 | 393 | 370 |
|  07 | 3,535 | 1,808 |
|  08 | 810 | 864 |
|  09 | 6,766 | 4,240 |
|  10 | 3,152 | 3,042 |
|  11 | 248 | 235 |
|  12 | 1,668 | 1,457 |
|  13 | 2,972 | 2,875 |
|  14 | 784 | 711 |
|  15 | 441 | 405 |
|  16 | 2,079 | 1,905 |
|  17 | 1,263 | 981 |
|  18 | 3,499 | 3,052 |
|  19 | 435 | 310 |
|  20 | 927 | 897 |
|  21 | 10,113 | 6,852 |
|  22 | 788 | 671 |
|  **Total** | **54,750** | **42,168** |


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