On Friday, July 19, 2013 4:11 AM Greg Smith wrote:
>On 7/9/13 12:09 AM, Amit Kapila wrote:
>>    I think the first thing to verify is whether the results posted can be 
>> validated in some other environment setup by another person.
>>    The testcase used is posted at below link:
>>    http://www.postgresql.org/message-id/51366323.8070...@vmware.com

>That seems easy enough to do here, Heikki's test script is excellent. 
>The latest patch Hari posted on July 2 has one hunk that doesn't apply 
>anymore now.

The Head code change from Heikki is correct.
During the patch rebase to latest PG LZ optimization code, the above code 
change is missed.

Apart from the above changed some more changes are done in the patch, those 

1. corrected some comments in the code 
2. Added a validity check as source and history length combined cannot be more 
than or equal to 8192. 

Thanks for the review, please find the latest patch attached in the mail.

Hari babu.

Attachment: pglz-with-micro-optimization-compress-using-newdata-3.patch
Description: Binary data

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to