search

SpineEventEngine / validation

Apache License 2.0
2 stars 0 forks source link

Generated message types should implement `MessageWithConstraints` #52

Open alexander-yevsyukov opened 1 year ago

alexander-yevsyukov commented 1 year ago

... at least for backward compatibility with the code from core-java.

alexander-yevsyukov commented 1 year ago

@armiol, it would partially solve the issue with ”required” INT fields we have in core-java.

If we do this, the checking of command messages would just call validate() once again.

armiol commented 1 year ago

We have discussed that with @dmdashenkov several weeks ago.

What I proposed is to omit this interface for now, since there is no code which ever uses this interface at runtime.

armiol commented 1 year ago

Let's discuss this issue vocally tomorrow.