spmallette commented on code in PR #3483: URL: https://github.com/apache/tinkerpop/pull/3483#discussion_r3553186119
########## docs/src/upgrade/release-4.x.x.asciidoc: ########## @@ -800,6 +887,37 @@ Kryo's default `Map` serializer because it extended `HashMap`, whereas in 4.x it serializer. As a result, Gryo-serialized `Tree` data written by 3.x cannot be read by 4.x and vice versa. The Gryo type registration id (`61`) is unchanged and 4.x-to-4.x Gryo round-trips correctly; this Gryo break is expected for the 4.0.0 major release. +===== Multi-Label Support Review Comment: Seems worth mentioning that the intent is that `getLabelCardinality` may be a static value if that's all the graph support but ultimately should reflect the state of the current graph instance. I'm not sure that's always clear to folks. There is a fair bit of detail here in the Upgrade docs for providers - possibly too much? Shouldn't we put these details in the Provider docs themselves and link to it from here with a more brief introduction? -- 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]
