On Mon, Jan 23, 2017 at 5:25 AM, Amit Langote
wrote:
> I tried implementing the second idea in the attached patch. It fixes the
> bug (multiple reports as mentioned in the commit message) that tuples may
> be inserted into the wrong partition.
Looks good to me,
On 2017/01/19 5:25, Robert Haas wrote:
> On Wed, Jan 11, 2017 at 10:53 PM, Amit Langote wrote:
>> On 2017/01/06 20:23, Amit Langote wrote:
>>>
>>> If a single BulkInsertState object is passed to
>>> heap_insert()/heap_multi_insert() for different heaps corresponding to
>>> different partitions
On Wed, Jan 11, 2017 at 10:53 PM, Amit Langote
wrote:
> On 2017/01/06 20:23, Amit Langote wrote:
>> On 2017/01/05 3:26, Robert Haas wrote:
>>> It's unclear to me why we need to do 0002. It doesn't seem like it
>>> should be necessary, it doesn't seem like a good