From cced9222e18ac985c7f1905b918468e5d7ec94b6 Mon Sep 17 00:00:00 2001 From: Andrew Hutchings Date: Wed, 10 May 2017 12:03:21 +0100 Subject: [PATCH] MCOL-710 Fix ALTER TABLE ADD COLUMN for TEXT/BLOB The column fill didn't detect that this was a TEXT/BLOB column. --- writeengine/wrapper/writeengine.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/writeengine/wrapper/writeengine.cpp b/writeengine/wrapper/writeengine.cpp index 81aab49ee..45447d08c 100644 --- a/writeengine/wrapper/writeengine.cpp +++ b/writeengine/wrapper/writeengine.cpp @@ -535,14 +535,20 @@ int WriteEngineWrapper::fillColumn(const TxnID& txnid, const OID& dataOid, //Bug 1703,1705 bool isToken = false; if (((dataType == CalpontSystemCatalog::VARCHAR) && (dataWidth > 7)) || - ((dataType == CalpontSystemCatalog::CHAR) && (dataWidth > 8)) || (dataType == CalpontSystemCatalog::VARBINARY) ) + ((dataType == CalpontSystemCatalog::CHAR) && (dataWidth > 8)) || + (dataType == CalpontSystemCatalog::VARBINARY) || + (dataType == CalpontSystemCatalog::BLOB) || + (dataType == CalpontSystemCatalog::TEXT)) { isToken = true; } Convertor::convertColType(dataType, newColType, isToken); if (((refColDataType == CalpontSystemCatalog::VARCHAR) && (refColWidth > 7)) || - ((refColDataType == CalpontSystemCatalog::CHAR) && (refColWidth > 8)) || (refColDataType == CalpontSystemCatalog::VARBINARY)) + ((refColDataType == CalpontSystemCatalog::CHAR) && (refColWidth > 8)) || + (refColDataType == CalpontSystemCatalog::VARBINARY) || + (dataType == CalpontSystemCatalog::BLOB) || + (dataType == CalpontSystemCatalog::TEXT)) { isToken = true; }