The patch is working fine and looks good to me however some code related review comments:
1. The test case is particularly written for Temporary table with Commit Drop, so please mention it in the comments.
2. Also, the class TestQueryUpdatableResultset is been inherited, so no need to repeat the same methods, for example: setUp, _execute_select_sql, tearDown
Hi Hackers, Attached is the patch to fix the Issue with temporary tables on commit drop as. This patch fixes the issue where an error was thrown when the user tried to create a temporary table via query tool and no result was displayed when the query was executed.
Please review.
Please write the API test cases for the same even if the changes are very small but they are in the SQL Editor backend code.
The patch is working fine and looks good to me.
Thanks,
Khushboo
--
Regards,
Amol Vyavahare
EnterpriseDB Corporation The Enterprise Postgres Company
T: +91 20 6644 9600, M: +91 738 738 2128
Unit 301 A, Third Floor, Godrej Castlemaine, 155, Sassoon Road, Pune, MH - 411001