gaborgsomogyi commented on pull request #30366: URL: https://github.com/apache/spark/pull/30366#issuecomment-728044045
I've had a deeper look and the title was a little bit misleading to me. I thought we intend to change the framework itself bit is seems like has effect only on HadoopFS provider. This makes the situation less invasive because the whole logic stays the same but some provider internals what we need to change. Though I think it would be include `HadoopFSDelegationTokenProvider` in the title. I think not filling such fields is a bug in the client itself but I feel the pain from user perspective. Without super deep consideration and since we're speaking about at least hours or days not yet sure why we're not just using `now` as issue date (with a super big warning printout)? I need to dig a little bit deeper to have good internals knowledge (I've mainly reviewed and developer the framework itself). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
