[GitHub] incubator-quickstep issue #320: Support Multiple Tuple Inserts

2017-10-27 Thread jianqiao 
Github user jianqiao commented on the issue:

https://github.com/apache/incubator-quickstep/pull/320
  
It may just be fine before `ExecutionGenerator`, I think the fix is to 
revise `ExecutionGenerator::convertInsertTuple()` and do some modifications 
inside `InsertOperator`.


---

[GitHub] incubator-quickstep issue #320: Support Multiple Tuple Inserts

2017-10-27 Thread hbdeshmukh 
Github user hbdeshmukh commented on the issue:

https://github.com/apache/incubator-quickstep/pull/320
  
I agree with @zuyu. We should reconsider the design. 


---