akhaku commented on code in PR #1635:
URL: 
https://github.com/apache/cassandra-java-driver/pull/1635#discussion_r1525623390


##########
core/src/main/java/com/datastax/oss/driver/internal/core/type/codec/UdtCodec.java:
##########
@@ -105,10 +105,7 @@ public UdtValue decode(@Nullable ByteBuffer bytes, 
@NonNull ProtocolVersion prot
       int i = 0;
       while (input.hasRemaining()) {
         if (i == cqlType.getFieldTypes().size()) {
-          throw new IllegalArgumentException(
-              String.format(
-                  "Too many fields in encoded UDT value, expected %d",
-                  cqlType.getFieldTypes().size()));
+          break;

Review Comment:
   It's been a while but my memory is a little fuzzy as to the exact scenario 
with which we ran into this, but it had to do with the object mapper caching 
schemas somewhere too. In any case, the client metadata refresh hint sounds 
reasonable for when we tackle what you said earlier about EVENT delivery 
failing, since that would be the logical place for the hint to propagate and 
affect. For now I'll add a debug log.



-- 
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: commits-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to