diff --git a/writeengine/wrapper/writeengine.cpp b/writeengine/wrapper/writeengine.cpp
index 2fefb8328..66708ce63 100644
--- a/writeengine/wrapper/writeengine.cpp
+++ b/writeengine/wrapper/writeengine.cpp
@@ -400,7 +400,7 @@ void WriteEngineWrapper::updateMaxMinRange(const size_t totalNewRow, const size_
                 break;
             }
             default:
-                idbassert(0 && "unknown WR type tag");
+                idbassert_s(0, "unknown WR type tag");
                 return;
         }
         if (maxMin->isBinaryColumn())
@@ -4588,8 +4588,6 @@ int WriteEngineWrapper::updateColumnRec(const TxnID& txnid,
         rc = writeColumnRecUpdate(txnid, cscColTypeList, colStructList, colValueList, colOldValueList,
                             ridLists[extent], tableOid, true, ridLists[extent].size(), &currentExtentRangesPtrs);
 
-//        m_opType = NOOP; // XXX: WHY WAS IT HERE???!!!
-
         if (rc != NO_ERROR)
             break;