1. I think that `Byte.valueOf(byte)` is meant for this;
2. I may be wrong, but I think that the only thing that can promote to `Byte` 
is `byte`, I don't believe you can cast boxed types, like `(Double)(Integer) 
getBlah()`;

Additionally, it looks like the first parameter will be boxed by 
`assertEquals(Object, Object)` anyway, so essentially with a cast (either 
`(Byte)` or `(Object)`, doesn't matter) or without a cast this should be 
equivalent to `assertTrue(Byte.valueOf(1).equals(nestedRow.getByte("aByte")))`, 
I may be missing something though.

Plus this test is not ensuring that it's actually a `Byte`, not `byte`. I would 
probably add a test that returns `null`.

[ Full content available at: https://github.com/apache/beam/pull/6383 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to