Skip to content

Conversation

@niloc132
Copy link
Member

No description provided.

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

No docs changes detected for 8b3562b

README.md Outdated
Comment on lines 19 to 28
```
A.B.C
A - Major version, incremented for breaking changes to the wire format
B - Minor version, incremented for backwards-compatible changes to schema
C - Patch version, incremented for backwards-compatible bug fixes
```

The minor version will not be reset to zero when the major version is incremented, to indicate relative compatibility
between two major releases for schema features. That is to say that the minor version is effectively a wire format
version.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if it would be good to give some examples and describe what is and is not compatible. Some of this wording here is confusing to me wrt what has been previously said. For example,

A - Major version, incremented for breaking changes to the wire format

seems at best incomplete, and potentially misleading without additional context from the rest of this document (that is, a FB bump also bumps the major version); and then it's still not clear to me what the intentions are.

For example, if we truly needed a breaking change to the wire format, what does that look like and how is it different from a "compatible" FB bump?

Would something like

A.B.C.D
A - breaking change wrt wire format
B - breaking change wrt using as library (FB bump implies bump here)
C - backwards-compatible changes to schema(s)
D - backwards-compatible bug fixes

make our versioning easier / more explicit? (I could also see the argument to switch A and B in this example.)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that probably matters a little more once we have such an example - a table would be good too of released versions, once we have something beyond 1.0.0?

I don't hate adding a fourth, but its not semver any more explicitly that way. This library is so thin I'm not even sure we can have "bugfixes" beyond doc typos?

niloc132 and others added 2 commits December 19, 2025 12:00
Co-authored-by: Charles P. Wright <cpwright@gmail.com>
devinrsmith
devinrsmith previously approved these changes Dec 19, 2025
@rcaudy rcaudy merged commit 5a485b8 into deephaven:main Dec 19, 2025
5 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants