Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write minimal columns #154

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Bengreen
Copy link

@Bengreen Bengreen commented Jun 9, 2016

When doing UPDATE operations the current API will read the whole record and then write a complete new record.
This update only reads the columns specifically requested and only writes the columns specifically requested so minimising back and forth to the foreign server.

Ben Greene added 4 commits June 9, 2016 00:18
… not every column

1. In multicornGetForeignRelSize use parsetree->targetList instead of baserel->reltargetlist to request a scan of only the requested columns, not all columns
1. In MulticornModifyState struct add update_columns_list to hold the list of columns to be updated
1. In PlanForeignModify return a list of the columns to be udpated by modify operation
1. In BeginForeignModify collect the list that was pushed to the modify operations and store it in fdw_private
1. In ExecForeignUpdate pass the column_update_list to the python object for use in update function
1. Update __init__ to add new method for 4 parameter update method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant