[ 
https://issues.apache.org/jira/browse/TEZ-1757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14220144#comment-14220144
 ] 

Prakash Ramachandran commented on TEZ-1757:
-------------------------------------------

+1 lgtm

columns.length and columns.@each are different. for ex if i replace a column 
entry length does not change, but columns.@each will fire. however since we 
always return a new columns array and not replacing column entry, current code 
looks good.

> Column selector for tables.
> ---------------------------
>
>                 Key: TEZ-1757
>                 URL: https://issues.apache.org/jira/browse/TEZ-1757
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Sreenath Somarajapuram
>            Assignee: Sreenath Somarajapuram
>         Attachments: TEZ-1757.1.patch, TEZ-1757.2.patch, TEZ-1757.3.patch, 
> TEZ-1757.wip.1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to