[GitHub] [druid] clintropolis commented on pull request #14334: use the latest datasketches-java-4.0.0

["clintropolis (via GitHub)" <gi...@apache.org>]

clintropolis commented on PR #14334:
URL: https://github.com/apache/druid/pull/14334#issuecomment-1560480542

   >It seems that some test failed, but I cannot reproduce that in my environment. I guess some expectations of rank should be adjusted, but that particular case is not exercised when I run "mvn test" for some reason.
   
   Looks like this is probably related to SQL compatible null handling mode, you can run locally by adding `-Ddruid.generic.useDefautlValueForNull=false`, it does not run by default but we do run both modes in CI.
   
   The other thing that needs updated is the licenses.yaml file entries for the dependencies which have been updated https://github.com/apache/druid/blob/master/licenses.yaml#L3787


-- 
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-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org