[MCOL-4732] WriteEngine code must use unsigned int to store column width Created: 2021-05-24  Updated: 2021-06-10

Status: Open
Project: MariaDB ColumnStore
Component/s: writeengine
Affects Version/s: 6.1.1
Fix Version/s: Icebox

Type: Bug Priority: Major
Reporter: Roman Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None


 Description   

WE uses int to store column width that contradicts with the fact that column width is always positive.
struct ColStruct defined in writeengine/shared/we_type.h must use unsigned uint32_t to store the width. This affects some interfaces in at least WE code. One needs to find the interfaces effected in WE and other potential consumers of the WE code, e.g. DMLProc, DDLProc.

 Comments   
Comment by David Hall (Inactive) [ 2021-05-24 ]

This will be superseded by MCOL-3879, which is changing colWidth to uint64_t

Comment by David Hall (Inactive) [ 2021-05-24 ]

This will be superseded by MCOL-3879, which is changing colWidth to uint64_t. As part of that effort all references to colWidth or about colWidth will be changed to uint64_t throughout the code.

Generated at Thu Feb 08 02:52:34 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.