-
Notifications
You must be signed in to change notification settings - Fork 12
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
main: FieldValue interface #75
Conversation
b9836fe
to
84e76f4
Compare
databind/src/main/java/tech/ydb/yoj/databind/expression/values/TupleFieldValue.java
Outdated
Show resolved
Hide resolved
databind/src/main/java/tech/ydb/yoj/databind/expression/values/BoolFieldValue.java
Outdated
Show resolved
Hide resolved
databind/src/main/java/tech/ydb/yoj/databind/expression/values/ByteArrayFieldValue.java
Outdated
Show resolved
Hide resolved
@lavrukov Please fix minor checkstyle failures:
and this will be ready to merge, I think :-) |
a49a077
to
0f8578f
Compare
This is impossible due to rules for |
UPD: Brought back compatibility |
@lavrukov Sasha, please look at this. I've made the (And we'll still release this as the next major version, 2.6.0) |
databind/src/main/java/tech/ydb/yoj/databind/expression/values/BooleanFieldValue.java
Show resolved
Hide resolved
databind/src/main/java/tech/ydb/yoj/databind/expression/ModelField.java
Outdated
Show resolved
Hide resolved
…ll `*FieldValue`s explicitly non-null. Rename: `NumberFieldValue` -> `IntegerFieldValue` to reflect `FieldValueType`'s name 1:1. Fix minor Checkstyle violations.
…st methods. Rename: `BoolFieldValue` -> `BooleanFieldValue` to reflect `FieldValueType`'s name 1:1.
…rearrange IllegalExpressionException subclasses for field value validation errors and make most of them public
…ationTest locally
ddcae93
to
7c5a96f
Compare
No description provided.