Loading storage/ndb/include/kernel/AttributeHeader.hpp +5 −5 Original line number Diff line number Diff line Loading @@ -52,8 +52,7 @@ public: // NOTE: in 5.1 ctors and init take size in bytes /** Initialize AttributeHeader at location aHeaderPtr */ static AttributeHeader& init(void* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize); static void init(Uint32* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize); /** Returns size of AttributeHeader (usually one or two words) */ Uint32 getHeaderSize() const; // In 32-bit words Loading Loading @@ -113,10 +112,11 @@ public: */ inline AttributeHeader& AttributeHeader::init(void* aHeaderPtr, Uint32 anAttributeId, void AttributeHeader::init(Uint32* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize) { return * new (aHeaderPtr) AttributeHeader(anAttributeId, aByteSize); AttributeHeader ah(anAttributeId, aByteSize); *aHeaderPtr = ah.m_value; } inline Loading storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp +3 −3 Original line number Diff line number Diff line Loading @@ -1957,9 +1957,8 @@ int Dbtup::interpreterNextLab(Signal* signal, Uint32 TdataForUpdate[3]; Uint32 Tlen; AttributeHeader& ah= AttributeHeader::init(&TdataForUpdate[0], TattrId, TattrNoOfWords << 2); AttributeHeader ah(TattrId, TattrNoOfWords << 2); TdataForUpdate[0]= ah.m_value; TdataForUpdate[1]= TregMemBuffer[theRegister + 2]; TdataForUpdate[2]= TregMemBuffer[theRegister + 3]; Tlen= TattrNoOfWords + 1; Loading @@ -1975,6 +1974,7 @@ int Dbtup::interpreterNextLab(Signal* signal, // Write a NULL value into the attribute /* --------------------------------------------------------- */ ah.setNULL(); TdataForUpdate[0]= ah.m_value; Tlen= 1; } int TnoDataRW= updateAttributes(req_struct, Loading storage/ndb/src/kernel/blocks/dbtup/DbtupRoutines.cpp +4 −6 Original line number Diff line number Diff line Loading @@ -818,9 +818,7 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, Tablerec* const regTabPtr) { Uint32 keyReadBuffer[MAX_KEY_SIZE_IN_WORDS]; Uint32 attributeHeader; TableDescriptor* attr_descr = req_struct->attr_descr; AttributeHeader* ahOut = (AttributeHeader*)&attributeHeader; AttributeHeader ahIn(*updateBuffer); Uint32 attributeId = ahIn.getAttributeId(); Uint32 attrDescriptorIndex = attributeId << ZAD_LOG_SIZE; Loading @@ -843,7 +841,7 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, ReadFunction f = regTabPtr->readFunctionArray[attributeId]; AttributeHeader::init(&attributeHeader, attributeId, 0); AttributeHeader attributeHeader(attributeId, 0); req_struct->out_buf_index = 0; req_struct->max_read = MAX_KEY_SIZE_IN_WORDS; req_struct->attr_descriptor = attrDescriptor; Loading @@ -852,12 +850,12 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, req_struct->xfrm_flag = true; ndbrequire((this->*f)(&keyReadBuffer[0], req_struct, ahOut, &attributeHeader, attributeOffset)); req_struct->xfrm_flag = tmp; ndbrequire(req_struct->out_buf_index == ahOut->getDataSize()); if (ahIn.getDataSize() != ahOut->getDataSize()) { ndbrequire(req_struct->out_buf_index == attributeHeader.getDataSize()); if (ahIn.getDataSize() != attributeHeader.getDataSize()) { jam(); return true; } Loading storage/ndb/src/kernel/blocks/dbutil/DbUtil.cpp +2 −3 Original line number Diff line number Diff line Loading @@ -1169,9 +1169,7 @@ DbUtil::prepareOperation(Signal* signal, PreparePtr prepPtr) /************************************************************** * Attribute found - store in mapping (AttributeId, Position) **************************************************************/ AttributeHeader & attrMap = AttributeHeader::init(attrMappingIt.data, attrDesc.AttributeId, // 1. Store AttrId AttributeHeader attrMap(attrDesc.AttributeId, // 1. Store AttrId 0); if (attrDesc.AttributeKeyFlag) { Loading Loading @@ -1200,6 +1198,7 @@ DbUtil::prepareOperation(Signal* signal, PreparePtr prepPtr) return; } } *(attrMappingIt.data) = attrMap.m_value; #if 0 ndbout << "BEFORE: attrLength: " << attrLength << endl; #endif Loading storage/ndb/src/ndbapi/NdbOperationDefine.cpp +6 −8 Original line number Diff line number Diff line Loading @@ -408,9 +408,8 @@ NdbOperation::getValue_impl(const NdbColumnImpl* tAttrInfo, char* aValue) return NULL; }//if }//if Uint32 ah; AttributeHeader::init(&ah, tAttrInfo->m_attrId, 0); if (insertATTRINFO(ah) != -1) { AttributeHeader ah(tAttrInfo->m_attrId, 0); if (insertATTRINFO(ah.m_value) != -1) { // Insert Attribute Id into ATTRINFO part. /************************************************************************ Loading Loading @@ -536,12 +535,11 @@ NdbOperation::setValue( const NdbColumnImpl* tAttrInfo, tAttrId = tAttrInfo->m_attrId; m_no_disk_flag &= (tAttrInfo->m_storageType == NDB_STORAGETYPE_DISK ? 0:1); const char *aValue = aValuePassed; Uint32 ahValue; if (aValue == NULL) { if (tAttrInfo->m_nullable) { AttributeHeader& ah = AttributeHeader::init(&ahValue, tAttrId, 0); AttributeHeader ah(tAttrId, 0); ah.setNULL(); insertATTRINFO(ahValue); insertATTRINFO(ah.m_value); // Insert Attribute Id with the value // NULL into ATTRINFO part. DBUG_RETURN(0); Loading Loading @@ -577,8 +575,8 @@ NdbOperation::setValue( const NdbColumnImpl* tAttrInfo, // Excluding bits in last word const Uint32 sizeInWords = sizeInBytes / 4; (void) AttributeHeader::init(&ahValue, tAttrId, sizeInBytes); insertATTRINFO( ahValue ); AttributeHeader ah(tAttrId, sizeInBytes); insertATTRINFO( ah.m_value ); /*********************************************************************** * Check if the pointer of the value passed is aligned on a 4 byte boundary. Loading Loading
storage/ndb/include/kernel/AttributeHeader.hpp +5 −5 Original line number Diff line number Diff line Loading @@ -52,8 +52,7 @@ public: // NOTE: in 5.1 ctors and init take size in bytes /** Initialize AttributeHeader at location aHeaderPtr */ static AttributeHeader& init(void* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize); static void init(Uint32* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize); /** Returns size of AttributeHeader (usually one or two words) */ Uint32 getHeaderSize() const; // In 32-bit words Loading Loading @@ -113,10 +112,11 @@ public: */ inline AttributeHeader& AttributeHeader::init(void* aHeaderPtr, Uint32 anAttributeId, void AttributeHeader::init(Uint32* aHeaderPtr, Uint32 anAttributeId, Uint32 aByteSize) { return * new (aHeaderPtr) AttributeHeader(anAttributeId, aByteSize); AttributeHeader ah(anAttributeId, aByteSize); *aHeaderPtr = ah.m_value; } inline Loading
storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp +3 −3 Original line number Diff line number Diff line Loading @@ -1957,9 +1957,8 @@ int Dbtup::interpreterNextLab(Signal* signal, Uint32 TdataForUpdate[3]; Uint32 Tlen; AttributeHeader& ah= AttributeHeader::init(&TdataForUpdate[0], TattrId, TattrNoOfWords << 2); AttributeHeader ah(TattrId, TattrNoOfWords << 2); TdataForUpdate[0]= ah.m_value; TdataForUpdate[1]= TregMemBuffer[theRegister + 2]; TdataForUpdate[2]= TregMemBuffer[theRegister + 3]; Tlen= TattrNoOfWords + 1; Loading @@ -1975,6 +1974,7 @@ int Dbtup::interpreterNextLab(Signal* signal, // Write a NULL value into the attribute /* --------------------------------------------------------- */ ah.setNULL(); TdataForUpdate[0]= ah.m_value; Tlen= 1; } int TnoDataRW= updateAttributes(req_struct, Loading
storage/ndb/src/kernel/blocks/dbtup/DbtupRoutines.cpp +4 −6 Original line number Diff line number Diff line Loading @@ -818,9 +818,7 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, Tablerec* const regTabPtr) { Uint32 keyReadBuffer[MAX_KEY_SIZE_IN_WORDS]; Uint32 attributeHeader; TableDescriptor* attr_descr = req_struct->attr_descr; AttributeHeader* ahOut = (AttributeHeader*)&attributeHeader; AttributeHeader ahIn(*updateBuffer); Uint32 attributeId = ahIn.getAttributeId(); Uint32 attrDescriptorIndex = attributeId << ZAD_LOG_SIZE; Loading @@ -843,7 +841,7 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, ReadFunction f = regTabPtr->readFunctionArray[attributeId]; AttributeHeader::init(&attributeHeader, attributeId, 0); AttributeHeader attributeHeader(attributeId, 0); req_struct->out_buf_index = 0; req_struct->max_read = MAX_KEY_SIZE_IN_WORDS; req_struct->attr_descriptor = attrDescriptor; Loading @@ -852,12 +850,12 @@ Dbtup::checkUpdateOfPrimaryKey(KeyReqStruct* req_struct, req_struct->xfrm_flag = true; ndbrequire((this->*f)(&keyReadBuffer[0], req_struct, ahOut, &attributeHeader, attributeOffset)); req_struct->xfrm_flag = tmp; ndbrequire(req_struct->out_buf_index == ahOut->getDataSize()); if (ahIn.getDataSize() != ahOut->getDataSize()) { ndbrequire(req_struct->out_buf_index == attributeHeader.getDataSize()); if (ahIn.getDataSize() != attributeHeader.getDataSize()) { jam(); return true; } Loading
storage/ndb/src/kernel/blocks/dbutil/DbUtil.cpp +2 −3 Original line number Diff line number Diff line Loading @@ -1169,9 +1169,7 @@ DbUtil::prepareOperation(Signal* signal, PreparePtr prepPtr) /************************************************************** * Attribute found - store in mapping (AttributeId, Position) **************************************************************/ AttributeHeader & attrMap = AttributeHeader::init(attrMappingIt.data, attrDesc.AttributeId, // 1. Store AttrId AttributeHeader attrMap(attrDesc.AttributeId, // 1. Store AttrId 0); if (attrDesc.AttributeKeyFlag) { Loading Loading @@ -1200,6 +1198,7 @@ DbUtil::prepareOperation(Signal* signal, PreparePtr prepPtr) return; } } *(attrMappingIt.data) = attrMap.m_value; #if 0 ndbout << "BEFORE: attrLength: " << attrLength << endl; #endif Loading
storage/ndb/src/ndbapi/NdbOperationDefine.cpp +6 −8 Original line number Diff line number Diff line Loading @@ -408,9 +408,8 @@ NdbOperation::getValue_impl(const NdbColumnImpl* tAttrInfo, char* aValue) return NULL; }//if }//if Uint32 ah; AttributeHeader::init(&ah, tAttrInfo->m_attrId, 0); if (insertATTRINFO(ah) != -1) { AttributeHeader ah(tAttrInfo->m_attrId, 0); if (insertATTRINFO(ah.m_value) != -1) { // Insert Attribute Id into ATTRINFO part. /************************************************************************ Loading Loading @@ -536,12 +535,11 @@ NdbOperation::setValue( const NdbColumnImpl* tAttrInfo, tAttrId = tAttrInfo->m_attrId; m_no_disk_flag &= (tAttrInfo->m_storageType == NDB_STORAGETYPE_DISK ? 0:1); const char *aValue = aValuePassed; Uint32 ahValue; if (aValue == NULL) { if (tAttrInfo->m_nullable) { AttributeHeader& ah = AttributeHeader::init(&ahValue, tAttrId, 0); AttributeHeader ah(tAttrId, 0); ah.setNULL(); insertATTRINFO(ahValue); insertATTRINFO(ah.m_value); // Insert Attribute Id with the value // NULL into ATTRINFO part. DBUG_RETURN(0); Loading Loading @@ -577,8 +575,8 @@ NdbOperation::setValue( const NdbColumnImpl* tAttrInfo, // Excluding bits in last word const Uint32 sizeInWords = sizeInBytes / 4; (void) AttributeHeader::init(&ahValue, tAttrId, sizeInBytes); insertATTRINFO( ahValue ); AttributeHeader ah(tAttrId, sizeInBytes); insertATTRINFO( ah.m_value ); /*********************************************************************** * Check if the pointer of the value passed is aligned on a 4 byte boundary. Loading