Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4338: test infra data migrator: include tables' primary keys in PostgreSQL ......................................................................
Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4951/2/tests/comparison/db_connection.py File tests/comparison/db_connection.py: PS2, Line 797: 3 why 3? http://gerrit.cloudera.org:8080/#/c/4951/2/tests/comparison/tests/test_cursor.py File tests/comparison/tests/test_cursor.py: PS2, Line 43: cllass class Line 172: def test_postgres_table_reading(postgresql_cursor, sql_primary_key_map): Are we connecting to a real Postgres instance in this test? If so, where do we configure the host and port, or do we just always connect to the local instance? -- To view, visit http://gerrit.cloudera.org:8080/4951 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I447f022e2dc3d4fc8373b7f388c7875a869921b8 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Michael Brown <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-HasComments: Yes
