You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@fzhedu: Before supporting left/right join, we should first support "null" value.
There are typically two methods to support null in a database system:
Assign one value as null for each data type. For instance, we can represent null in unsigned long as the maximum value. This method simplifies the implementation, but the value representing null cannot be used as a normal value.
Use specific bit to mask whether a value is null.
In our system, as all tuples within a table share one schema, there is no additional schema information for each tuple. Consequently, supporting null value under the second method will need a complete rewrite of Claims and violent the design philosophy of data reorganization in Claims. For implementation simplicity and the performance concern, I believe we should take the first method.
@scdong: Could you please handle this enhancement.
To support null, I think we should
add a boolean in data_type to represent whether the type is nullable or not.
support setNull(), isNull() in operate class.
Also toValue() should prints "NULL" when the data type is nullable and the value to print represents "null".
The text was updated successfully, but these errors were encountered:
@fzhedu: Before supporting left/right join, we should first support "null" value.
There are typically two methods to support null in a database system:
In our system, as all tuples within a table share one schema, there is no additional schema information for each tuple. Consequently, supporting null value under the second method will need a complete rewrite of Claims and violent the design philosophy of data reorganization in Claims. For implementation simplicity and the performance concern, I believe we should take the first method.
@scdong: Could you please handle this enhancement.
To support null, I think we should
The text was updated successfully, but these errors were encountered: