Skip to content

Conversation

@chanukya571
Copy link

Hey All,

Sorry for delay. there is a problem with my email.

As mentioned I have reviewed the code and did some cleanup , added comments also.

As per suggestions on PR reorg#327 I have looked in both repack_one_table() and repack_table_indexes() but the ERROR: relation pg_class does not exist is coming from is_requested_relation_exists() before checking the function check_systemtables().

So I have added check_systemtables() in is_requested_relation_exists() so that it will be clear for understanding and enhance in future.

Please let me know your thoughts. once this branch is merged I will create pull request to reorg/pg_repack.

Thanks.

@chanukya571
Copy link
Author

@za-arthur Please review this. Once this is merged I will create a Pull request from adjust/pg_repack to reorg/pg_repack.
Or let me know if I have to create pull directly from my fork with reorg/pg_repack.

Thanks

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