Ma77Ball opened a new pull request, #5030:
URL: https://github.com/apache/texera/pull/5030
### What changes were proposed in this PR?
`ArrowUtils.toAttributeType` (and the mirror dispatch in `setTexeraTuple`)
used `case 64 | _ => LONG` for `ArrowType.Int`. The trailing `_` made the
second arm a catch-all, so every Int width that was not 16 or 32 (e.g. 8,
128) silently surfaced as `LONG` rather than being rejected. This PR
splits the alternation so 64 stays `LONG`, 16/32 stay `INTEGER`, and any other
width throws `AttributeTypeException`, matching how every other unsupported
ArrowType is already handled in the same method.
### Any related issues, documentation, or discussions?
Closes: #4760
### How was this PR tested?
* Updated the existing pin test in `ArrowUtilsSpec` (which previously
documented the silent coercion and invited this fix) to assert that `Int(8)`
and `Int(128)` now raise `AttributeTypeException`.
* `sbt scalafmtAll` clean.
* `sbt "workflowCore/testOnly
org.apache.texera.amber.util.ArrowUtilsSpec"` expected to pass; operator-side
`ArrowUtilsSpec` only exercises widths 16/32/64 and is unaffected.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.7 in compliance with ASF
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
