Address reviewer comments

Author: guilhermeleobas

doc/api.rst

@@ -60,7 +60,7 @@ Externals
 
 
 HeavyDB Backend
-=================
+===============
 
 The table below contains the data structures available for the HeavyDB backend.
 It should be noticed that the following types are not regular Python types but

rbc/heavyai/text_encoding_none.py

@@ -1,4 +1,4 @@
-'''Omnisci TextEncodingNone type that corresponds to Omnisci type TEXT ENCODED NONE.
+'''HeavyDB TextEncodingNone type that corresponds to HeavyDB type TEXT ENCODED NONE.
 '''
 
 __all__ = ['TextEncodingNonePointer', 'TextEncodingNone', 'HeavyDBTextEncodingNoneType']
@@ -8,15 +8,15 @@
 from rbc.targetinfo import TargetInfo
 from rbc.errors import RequireLiteralValue
 from .buffer import (
-    BufferPointer, Buffer, OmnisciBufferType,
-    omnisci_buffer_constructor)
+    BufferPointer, Buffer, HeavyDBBufferType,
+    heavydb_buffer_constructor)
 from numba.core import types, extending, cgutils
 from llvmlite import ir
 from typing import Union
 
 
-class HeavyDBTextEncodingNoneType(OmnisciBufferType):
-    """Omnisci TextEncodingNone type for RBC typesystem.
+class HeavyDBTextEncodingNoneType(HeavyDBBufferType):
+    """HeavyDB TextEncodingNone type for RBC typesystem.
     """
 
     @property
@@ -118,20 +118,20 @@ def impl(a, b):
 
 
 @extending.lower_builtin(TextEncodingNone, types.Integer)
-def omnisci_text_encoding_none_constructor(context, builder, sig, args):
-    return omnisci_buffer_constructor(context, builder, sig, args)
+def heavydb_text_encoding_none_constructor(context, builder, sig, args):
+    return heavydb_buffer_constructor(context, builder, sig, args)
 
 
 @extending.lower_builtin(TextEncodingNone, types.StringLiteral)
-def omnisci_text_encoding_none_constructor_literal(context, builder, sig, args):
+def heavydb_text_encoding_none_constructor_literal(context, builder, sig, args):
     int64_t = ir.IntType(64)
     int8_t_ptr = ir.IntType(8).as_pointer()
 
     literal_value = sig.args[0].literal_value
     sz = int64_t(len(literal_value))
 
     # arr = {ptr, size, is_null}*
-    arr = omnisci_buffer_constructor(context, builder, sig.return_type(types.int64), [sz])
+    arr = heavydb_buffer_constructor(context, builder, sig.return_type(types.int64), [sz])
     ptr = builder.extract_value(builder.load(arr), [0])
 
     msg_bytes = literal_value.encode('utf-8')
@@ -144,7 +144,7 @@ def omnisci_text_encoding_none_constructor_literal(context, builder, sig, args):
 
 
 @extending.type_callable(TextEncodingNone)
-def type_omnisci_text_encoding_none(context):
+def type_heavydb_text_encoding_none(context):
     def typer(arg):
         if isinstance(arg, types.UnicodeType):
             raise RequireLiteralValue()

rbc/tests/heavyai/test_text_encoding.py

@@ -227,7 +227,6 @@ def test_text_encoding_column_list2(heavydb, size):
 @pytest.mark.usefixtures("create_columns")
 @pytest.mark.parametrize("size", (32,))
 @pytest.mark.parametrize("num_cols", (1, 2, 3, 4))
-@pytest.mark.skip()
 def test_text_encoding_column_list3(heavydb, size, num_cols):
     heavydb.require_version((5, 7), "Requires heavydb-internal PR 5719")