churromorales commented on PR #13804:
URL: https://github.com/apache/druid/pull/13804#issuecomment-1489512690

   @abhishekagarwal87 The test failing are from the other PR that was merged, I 
don't have cycles to look and fix whatever is wrong and we honestly forked our 
druid so whether it goes upstream is not a big deal.  
   
   I think the thing is the other PR is using all the old libraries, and it 
can't deserialize the pod template anymore.  So if you want to update this 
feature to work with newer k8s versions, that will have to be sorted out at 
some point, or you can just close off this PR and worry about upgrades when the 
time comes.  As it stands the mm-less wont work in newer versions of k8s so, 
I'll leave it up to imply what they want to do.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to