leekeiabstraction commented on code in PR #2312:
URL: https://github.com/apache/fluss/pull/2312#discussion_r2868003390


##########
fluss-common/src/test/java/org/apache/fluss/row/encode/CompactedKeyEncoderTest.java:
##########
@@ -154,6 +171,22 @@ void testGetKeyForAllTypes() throws Exception {
 
             InternalRow keyRow = decodeRow(keyDataTypes, keyBytes);
 
+            URL url = 
getClass().getClassLoader().getResource(ENCODED_KEY_HEX_FILE_PATH);

Review Comment:
   Ack, just to clarify, you're referring to loading the file from test 
resources is too complex right?
   
   In that case I'll keep the same assertions, defining the data on Java source 
file. 



-- 
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]

Reply via email to