shivendra-dev54 opened a new pull request, #811:
URL: https://github.com/apache/incubator-graphar/pull/811
### Reason for this PR
The `graphar::Type` enum is part of public APIs and is also used in
serialization / layout-sensitive code paths. Previously, its underlying
type was implicit, which makes the width platform-dependent.
This PR makes the underlying type explicit so that it is stable and
well-defined across platforms.
### What changes are included in this PR?
- Update `graphar::Type` to use an explicit `int32_t` underlying type
- Align the forward declaration in `fwd.h` with the enum definition
No behavior changes are intended.
### Testing
- Code review and static inspection
- Local CMake configure succeeded
- Full build was not completed locally due to Arrow Acero/ORC header
availability issues in the conda-forge Arrow version
- Relying on CI for full build and integration testing
### Are there any user-facing changes?
No. This change is internal and does not affect file formats or API behavior.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
