Skip to content

Conversation

@marxhxxx
Copy link

@marxhxxx marxhxxx commented Feb 2, 2026

Summary: Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013

@meta-codesync
Copy link

meta-codesync bot commented Feb 2, 2026

@marxhxxx has exported this pull request. If you are a Meta employee, you can view the originating Diff in D92099013.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 2, 2026
@netlify
Copy link

netlify bot commented Feb 2, 2026

👷 Deploy Preview for meta-velox processing.

Name Link
🔨 Latest commit fa7516a
🔍 Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/69839a4ee1aa870008b6f27d

@marxhxxx marxhxxx changed the title feat: support unknown in coerceTypeBase feat: Support unknown in coerceTypeBase Feb 2, 2026
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 3, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 3, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 4, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 4, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 4, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost. Coercing UNKNOWN to UNKNOWN returns std::nullopt since no actual coercion is needed.

Differential Revision: D92099013
marxhxxx pushed a commit to marxhxxx/velox that referenced this pull request Feb 4, 2026
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to any target type with zero cost.

Differential Revision: D92099013
Summary:

Add support for UNKNOWN type coercion in coerceTypeBase. When the source type is UNKNOWN, it can be coerced to following types `TINYINT(), BOOLEAN(), SMALLINT(), INTEGER(), BIGINT(), REAL(), DOUBLE(), VARCHAR(), VARBINARY()`. Cost of the coercion is higher according to the list order.

Differential Revision: D92099013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant