[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15369825#comment-15369825
]
Hudson commented on YARN-4025:
--
SUCCESS: Integrated in Hadoop-trunk-Commit #10074 (See
[https
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14703465#comment-14703465
]
Sangjin Lee commented on YARN-4025:
---
Thanks everyone!
> Deal with byte representations o
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14702274#comment-14702274
]
Junping Du commented on YARN-4025:
--
bq. We can treat that method as the "default" readResu
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14702212#comment-14702212
]
Hadoop QA commented on YARN-4025:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701672#comment-14701672
]
Sangjin Lee commented on YARN-4025:
---
Oh OK. Got it. I thought you meant the line you refe
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701656#comment-14701656
]
Junping Du commented on YARN-4025:
--
bq. The EntityTable.java file is already fixed in the
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701645#comment-14701645
]
Sangjin Lee commented on YARN-4025:
---
Thanks for your review [~djp]!
{quote}
Do we handle
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701325#comment-14701325
]
Junping Du commented on YARN-4025:
--
bq. One major change I did is that now TestHBaseTimeli
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701309#comment-14701309
]
Junping Du commented on YARN-4025:
--
Thanks [~sjlee0] for updating the patch. 003 patch loo
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14700628#comment-14700628
]
Hadoop QA commented on YARN-4025:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14700524#comment-14700524
]
Li Lu commented on YARN-4025:
-
Latest patch LGTM. Thanks [~sjlee0]!
> Deal with byte represent
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1472#comment-1472
]
Sangjin Lee commented on YARN-4025:
---
That's a good point [~gtCarrera9]. Let me see if I c
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14697545#comment-14697545
]
Li Lu commented on YARN-4025:
-
Oh sorry I missed that line... That looks fine.
> Deal with by
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14697534#comment-14697534
]
Vrushali C commented on YARN-4025:
--
I changed it from '?' to '='. Sangjin was also wonder
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14697531#comment-14697531
]
Vrushali C commented on YARN-4025:
--
Hmm, yes I think some more comments there might help (
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14696271#comment-14696271
]
Li Lu commented on YARN-4025:
-
Oh and, BTW, why are we changing Separator.VALUES?
> Deal with
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14696268#comment-14696268
]
Li Lu commented on YARN-4025:
-
Hi [~sjlee0], thanks for the work! The patch overall LGTM. One s
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14694840#comment-14694840
]
Hadoop QA commented on YARN-4025:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14694177#comment-14694177
]
Vrushali C commented on YARN-4025:
--
Reassigning to Sangjin
> Deal with byte representati
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14692511#comment-14692511
]
Vrushali C commented on YARN-4025:
--
Yes, +1
> Deal with byte representations of Longs in
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14692468#comment-14692468
]
Sangjin Lee commented on YARN-4025:
---
For the record, we will go ahead with YARN-3906 firs
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680450#comment-14680450
]
Sangjin Lee commented on YARN-4025:
---
No, not really. I just wanted to understand the situ
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680430#comment-14680430
]
Vrushali C commented on YARN-4025:
--
Thanks [~sjlee0] and [~gtCarrera9]
I will rebase and
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680410#comment-14680410
]
Sangjin Lee commented on YARN-4025:
---
Went over the patch pretty quickly, and have some hi
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680393#comment-14680393
]
Li Lu commented on YARN-4025:
-
Hi [~sjlee0][~vrushalic], I slightly prefer to firstly move with
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14680388#comment-14680388
]
Sangjin Lee commented on YARN-4025:
---
Thanks for the patch [~vrushalic]. I believe you nee
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14662869#comment-14662869
]
Hadoop QA commented on YARN-4025:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-4025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14662833#comment-14662833
]
Vrushali C commented on YARN-4025:
--
The function {code} public Map readResults(Result resu
28 matches
Mail list logo