[
https://issues.apache.org/jira/browse/APEXCORE-482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15380063#comment-15380063
]
ASF GitHub Bot commented on APEXCORE-482:
-----------------------------------------
Github user vrozov commented on the issue:
https://github.com/apache/apex-core/pull/358
What is the reason unifier does not have it's own StreamMeta? IMO, it will
be better to introduce separate StreamMeta for unifiers instead of the locality
field that is used only for unifiers.
> Default Unifiers should be ThreadLocal with downstream operators
> ----------------------------------------------------------------
>
> Key: APEXCORE-482
> URL: https://issues.apache.org/jira/browse/APEXCORE-482
> Project: Apache Apex Core
> Issue Type: Improvement
> Reporter: Sandesh
> Assignee: Sandesh
>
> Default Unifiers are currently deployed ( except in Mx1 ) as CONTAINER_LOCAL
> with the downstream operator. It is better to make it THREAD_LOCAL to improve
> the performance.
> Custom unifiers can still be CONTAINER_LOCAL and possibly allow the users to
> control the behaviours.
> Note: Created a prototype to understand the root cause :
> https://github.com/sandeshh/apex-core/commit/550f31c770b704360c813924e85a9fffeaec7c53
> During PhysicalPlan creation downstream operator's input port won't have
> stream locality assigned. So StreamingContainerAgent assigns CONTAINER_LOCAL
> Actual implementation should be done in Physical Plan.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
