[GitHub] incubator-quickstep pull request #129: Partial inserts

2016-11-22 Thread cramja
Github user cramja closed the pull request at: https://github.com/apache/incubator-quickstep/pull/129 --- 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 fe

[GitHub] incubator-quickstep pull request #129: Partial inserts

2016-11-17 Thread cramja
Github user cramja commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/129#discussion_r88559668 --- Diff: storage/StorageBlockLayout.cpp --- @@ -154,11 +154,7 @@ StorageBlockLayout* StorageBlockLayout::GenerateDefaultLayout(const CatalogRelat

[GitHub] incubator-quickstep pull request #129: Partial inserts

2016-11-17 Thread cramja
Github user cramja commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/129#discussion_r88500702 --- Diff: storage/StorageBlock.hpp --- @@ -307,13 +307,60 @@ class StorageBlock : public StorageBlockBase { *iteration will be adva

[GitHub] incubator-quickstep pull request #129: Partial inserts

2016-11-17 Thread cramja
Github user cramja commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/129#discussion_r88499673 --- Diff: storage/InsertDestination.hpp --- @@ -517,6 +527,12 @@ class PartitionAwareInsertDestination : public InsertDestination { Val

[GitHub] incubator-quickstep pull request #129: Partial inserts

2016-11-07 Thread cramja
GitHub user cramja opened a pull request: https://github.com/apache/incubator-quickstep/pull/129 Partial inserts This change enables use of the PartialBulkInserts feature, an efficient implementation of which was introduced in the SplitRowStore in #100 . Most of these changes were