Noemi Pap-Takacs has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/21084 )
Change subject: IMPALA-12717: Fix error message for missing table descriptor ...................................................................... IMPALA-12717: Fix error message for missing table descriptor The error messages for missing table descriptor were malformed due to incorrect stringstream usage. The message was given in the constructor and the table id overwrote the beginning the following way: 'ERROR: 0ailed to get table descriptor for table id: ' The solution is not passing any string to the std::stringstream constructor and using only operator<< instead. Testing: Built a version locally where the table descriptor was missing from Iceberg DELETE statement, and verified that the error message is 'ERROR: Failed to get table descriptor for table id: 0'. Change-Id: Iae94efe7f60a2de7f4d9ec47ad9d2c1ee8ad0b41 --- M be/src/exec/hdfs-table-sink.cc M be/src/exec/iceberg-delete-sink-base.cc 2 files changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/84/21084/3 -- To view, visit http://gerrit.cloudera.org:8080/21084 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iae94efe7f60a2de7f4d9ec47ad9d2c1ee8ad0b41 Gerrit-Change-Number: 21084 Gerrit-PatchSet: 3 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Peter Rozsa <pro...@cloudera.com>