bkonold commented on issue #1113: SAMZA-2280: LocalApplicationRunner should use final, planned set of configs URL: https://github.com/apache/samza/pull/1113#issuecomment-517077038 Discussed offline. Though there is effectively no change in control flow, this PR brings the standalone case slightly closer to parity with YARN in that the ApplicationDescriptor will be constructed using the planned set of configs. I've additionally (in the latest commit) removed some redundancy between config parameters and application descriptor, to make it more clear to the reader what configs are being used throughout.
---------------------------------------------------------------- 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] With regards, Apache Git Services
