Miscellaneous minor changes 1) Revert a part of previous change to ComTdbHbaseAccess::displayRowId() for SHOWPLAN. Each valid row id should have 2 bytes of length followed by the value of id, if it is not missing or null. We need to treat the id as binary value so we'll print each id byte as 2-digit hex value up to 20 bytes.
2) Remove the part of comments in sql/sqlcomp/DefineConstants.h that requires not to delete and reuse existing constants. This is no longer mandatory in Trafodion.
3) Enable NA memory overflow checking when running executor and qat suites in debug flavor. This would increase the run time for these two suites by about 10%.
Rework according to review comments. Explanations above changed too.
SHOWPLAN related changes Reworked the fix for bug 1392522 - mxosrvr core dumped doing showplan (with N.E. enabled), plus other related code changes. 1) The native code (also known as native expression), if generated, is stored in the expression's constant area. SHOWPLAN will dump the native code in the assembly format. The display can be disabled by CQD PCODE_NE_IN_SHOWPLAN to "OFF". It is "ON" by default. This part had been reviewed by Jim Capps and Mike Hanlon. 2) Add several SHOWPLAN statements in core/TEST019 without logging the output. This is to ensure no core generated when getting executor operator (TDBs) info via SHOWPLAN. 3) A temporary fix to ComTdbHbaseAccess::displayRowId(). The current way of retrieving begin or end row IDs from the HbaseScanRows for SHOWPLAN does not match with the way those row IDs are generated (see HbaseAccess::genListsOfRows), causing core dumps in some cases.