dcapwell commented on PR #2310: URL: https://github.com/apache/cassandra/pull/2310#issuecomment-1589668257
For flaky test `RandomSchemaTest` `seed=1686651894317L` ``` Caused by: java.lang.AssertionError: Invalid value for row 0 column 6 (o of type 'org.apache.cassandra.db.marshal.DynamicCompositeType(r=>org.apache.cassandra.db.marshal.InetAddressType,N=>org.apache.cassandra.db.marshal.TimeType)'), expected <0x804e000800004a571799e37c00807200101488cd7435e195b091c85e26ecb2103600> (34 bytes) but got <0x000000030000003c00000001000000340000000300000010ecae961ff66edfec02f9079a38cebfb5000000040ede6ff2000000103a22603a0c8f2b57fe37a15028a39f120000006c000000030000003400000003000000109767762f06a54e754eab0b6b9429b29000000004dbd742b800000010a77459a2afb1ad22b562c378d51dbcf70000001c000000030000000428da1844000000045bcfe66400000004eb95981f0000000c0000000100000004a7ac7f7f0000001c000000010000001400000002000000044e9fcbfb00000004bc409874> (212 bytes) (using protocol version 5/v5) ``` I need to look into what happened here... InetAddress worst case is 16bytes (ipv6) and TimeType is 8 bytes... so 34 bytes makes sense... 212 does not! This has to have the type names in the encoding for some reason for this to make any sense to me... but even then... `org.apache.cassandra.db.marshal.InetAddressType` = `47` chars, `org.apache.cassandra.db.marshal.TimeType` = `40` chars... so 43 + 47 + 40 + 2 * 2 ~= 140 bytes... but we have 212! ``` Caused by: org.apache.cassandra.exceptions.InvalidRequestException: Key length of 68003 is longer than maximum of 65535 ``` yay random generation... this is clustering size check, with the following clustering key.. ``` "fu74dgMGHmnxwvi9a6m2zRGeFinFcSUZKleN6gKhFsRe5OQFD3" double, "W7X8WGc4HkJUyZVrg7Q" frozen<map<vector<frozen<map<text, text>>, 3>, frozen<map<boolean, frozen<tuple<'LexicalUUIDType', double, smallint>>>>>> ``` my guess is this is the map and not the double ^_^. Changed max depth to be 1 (was 2), so `map` can still have a `vector`, but the `vector` no longer allows a `map` and requires a primitive... this should lower the size without 100% remove testing nested support.... -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
