Log In - Home Page

Patch: Speed dblink using alternate libpq tuple storage

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2012-01
Topic Performance
Patch Status Committed
Author Kyotaro Horiguchi
Reviewers Marko Kreen
Committer Tom Lane
Close Date 2012-04-04
Comments
Patch by gsmith on 2012-01-14 06:34:29 AM: Initial version.
Patch by horiguti on 2012-01-17 08:59:28 AM: Renamed the terms in libpq patch, refined the error handling in dblink.
Review by markokr on 2012-01-20 03:21:19 PM: Request better API
Patch by horiguti on 2012-01-27 08:58:53 AM: New API is implemented.
Review by markokr on 2012-01-27 05:22:52 PM: Request minor cleanups.
Patch by horiguti on 2012-01-30 09:15:12 AM: New revision of the patch, and the result of performance test.
Comment by horiguti on 2012-01-30 11:40:43 AM: Sorry, dblink.c in this patch is broken in error handling. Re-issue later.
Review by markokr on 2012-01-30 06:22:08 PM: More notes.
Patch by horiguti on 2012-01-31 03:00:46 AM: This is the fixed version of dblink.c for row processor.
Patch by markokr on 2012-02-01 03:55:01 PM: Various cleanups and fixes.
Patch by horiguti on 2012-02-02 07:52:48 AM: The bug of realloc() usage is fixed.
Patch by markokr on 2012-02-02 02:41:43 PM: Ready for committer.
Patch by markokr on 2012-02-07 02:50:44 PM: Remove dynamic array, doc fixes.
Patch by markokr on 2012-02-14 12:30:09 AM: Skip row on exception, early exit possibility, minor cleanups.
Patch by horiguti on 2012-02-16 08:53:06 AM: PQskipRemainingResult() added and use it in dblink. PG_TRY() will be done once per query.
Review by markokr on 2012-02-16 07:02:49 PM: Request more general PGsetRowProcessorErrMsg() instead.
Comment by markokr on 2012-02-18 10:58:09 PM: Demos for new API
Patch by horiguti on 2012-02-21 03:41:27 PM: PQskipResult() added.
Patch by horiguti on 2012-02-23 10:59:02 AM: Release result from PQgetResult() in PQskipResult(). const'ify msg of PQsetRowProcessorErrMsg(). Change parameter order of PQrowProcessor.
Patch by horiguti on 2012-02-24 10:56:38 AM: Remove PGresult.rowProcessorErrMes, Renamed PQsetRowProcessorErrMes() to PQresultSetErrMes(), some cleansing of error handling in getAnotherTuple().
Patch by markokr on 2012-02-27 01:09:41 AM: Additional patch for PQgetRow() & PQrecvRow()
Patch by horiguti on 2012-02-28 08:06:56 AM: Restore the OOM route.
Patch by horiguti on 2012-03-07 06:23:57 AM: New version patch. Removed custom error message, Remove EOF return in getAnotherTuple@V3. Remove non-blocking check for return 2 of row processor.
Patch by markokr on 2012-03-07 08:35:20 PM: Final review & cleanups. Ready for committer v2.
Patch by horiguti on 2012-03-26 11:32:59 AM: Separate bug fix of original code from functional augmentation. Use palloc. Restore broken process. Change memory allocation policy.
Patch by horiguti on 2012-03-27 02:28:18 AM: Bug fix of the previous dblink patch.
Comment by markokr on 2012-03-30 04:35:11 PM: PQgetRow() API overview
Patch by horiguti on 2012-04-04 05:31:27 PM: Use old way for a async query with dblink_is_busy. Some bugs fixed. And rebased.

Add Comment

Please log in to comment on this patch.