nsivabalan commented on PR #19205:
URL: https://github.com/apache/hudi/pull/19205#issuecomment-5016823660

   CI caught one more null-vs-empty-string test I missed in the last pass — 
same class of fix as the earlier `TestHoodieSparkSqlWriter` update. 
`TestHoodieDatasetBulkInsertHelper.testBulkInsertHelperNoMetaFields` NPE'd on 
`entry.get(...).equals("")` because our nullable-stub change now writes null 
instead of an empty string.
   
   Switched the assertions from `.get(...).equals("")` to 
`entry.isNullAt(...)`, folded into the row-writer nullable-stub commit. Also 
re-rebased #19206 and the local phase-3 branch onto the updated head so they 
pick up the fix.


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