-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HUDI-6542] Fix ExpressionEvaluators doesnt support BIGINT #9215
[HUDI-6542] Fix ExpressionEvaluators doesnt support BIGINT #9215
Conversation
@danny0405 the same as #9112 |
667b0b5
to
94ddd4f
Compare
public static final DataType DATA_TYPE_BIGINT_PK = DataTypes.ROW( | ||
DataTypes.FIELD("uuid", DataTypes.BIGINT()), | ||
DataTypes.FIELD("name", DataTypes.VARCHAR(10)), | ||
DataTypes.FIELD("age", DataTypes.INT()), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we move the data types to TestConfigurations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved
94ddd4f
to
af80667
Compare
af80667
to
c235bc0
Compare
@hudi-bot run azure |
@danny0405 if there are no any questions it could be merged |
Change Logs
Add support of BIGINT type in ExpressionEvaluators
Impact
Bug fix
Risk level (write none, low medium or high below)
none
Documentation Update
No needed
Contributor's checklist