Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TEZ-4137: Input/Output/Processor should merge payload to local conf #64

Closed
wants to merge 1 commit into from

Conversation

mustafaiman
Copy link
Contributor

Runtime components do not have access to session config. Because of that, we have to send all config options to every task eventhough a large part of the configs are in session config, just not accessible to the task. This patch makes it accessible to the tasks. All input/output/processor s can merge their user payload config to this session config. Therefore, we can send less bytes over the wire, saving time in serializing and transmitting configs client-to-am and am-to-task communication.

Change-Id: I8cc0d17deec1e03793b884fd05460e3e69afab6a

Runtime components do not have access to session config. Because of that, we have to send all config options to every task eventhough a large part of the configs are in session config, just not accessible to the task. This patch makes it accessible to the tasks. All input/output/processor s can merge their user payload config to this session config. Therefore, we can send less bytes over the wire, saving time in serializing and transmitting configs client-to-am and am-to-task communication.

Change-Id: I8cc0d17deec1e03793b884fd05460e3e69afab6a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant