Removing check that offsets larger than INT32 are not allowed.

kevkrist · kevkrist · commit ae473ce133b2 · 2025-06-10T16:46:32.000Z
diff --git a/src/expression_executor/specializations/gpu_execute_reference.cpp b/src/expression_executor/specializations/gpu_execute_reference.cpp
@@ -28,12 +28,6 @@ std::unique_ptr<cudf::column> GpuExpressionExecutor::Execute(const BoundReferenc
     }
     return GpuDispatcher::DispatchMaterialize(input_column.get(), resource_ref, execution_stream);
   }
-  if (input_column->data_wrapper.type.id() == GPUColumnTypeId::VARCHAR &&
-      input_column->data_wrapper.num_bytes > INT32_MAX)
-  {
-    throw NotImplementedException(
-      "Execute[Reference]: string offsets larger than int32_t are not supported in libcudf");
-  }
   if (input_column->column_length > INT32_MAX)
   {
     throw NotImplementedException(

Original file line number	Diff line number	Diff line change
`@@ -28,12 +28,6 @@ std::unique_ptr<cudf::column> GpuExpressionExecutor::Execute(const BoundReferenc`
`28`	`28`	`}`
`29`	`29`	`return GpuDispatcher::DispatchMaterialize(input_column.get(), resource_ref, execution_stream);`
`30`	`30`	`}`
`31`		`- if (input_column->data_wrapper.type.id() == GPUColumnTypeId::VARCHAR &&`
`32`		`- input_column->data_wrapper.num_bytes > INT32_MAX)`
`33`		`- {`
`34`		`- throw NotImplementedException(`
`35`		`- "Execute[Reference]: string offsets larger than int32_t are not supported in libcudf");`
`36`		`- }`
`37`	`31`	`if (input_column->column_length > INT32_MAX)`
`38`	`32`	`{`
`39`	`33`	`throw NotImplementedException(`