[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15369761#comment-15369761
]
Hudson commented on YARN-3049:
--
SUCCESS: Integrated in Hadoop-trunk-Commit #10074 (See
[https
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14662068#comment-14662068
]
Junping Du commented on YARN-3049:
--
+1. Patch LGTM. [~sjlee0], please feel free to go ahea
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14662060#comment-14662060
]
Sangjin Lee commented on YARN-3049:
---
Let me know if there is any additional comments. I'l
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14661313#comment-14661313
]
Vrushali C commented on YARN-3049:
--
Filed https://issues.apache.org/jira/browse/YARN-4025
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14661012#comment-14661012
]
Sangjin Lee commented on YARN-3049:
---
Yes, +1 with proceeding with this patch and addressi
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14661007#comment-14661007
]
Li Lu commented on YARN-3049:
-
I checked EntityRowKey.java and seems like we never convert flow
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660991#comment-14660991
]
Vrushali C commented on YARN-3049:
--
bq. I'm worried that Bytes.toString() doesn't make the
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660987#comment-14660987
]
Vrushali C commented on YARN-3049:
--
Yes I will take that jira up.
> [Storage Implementa
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660952#comment-14660952
]
Zhijie Shen commented on YARN-3049:
---
As the issue is not blocking the whole reader implem
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660935#comment-14660935
]
Li Lu commented on YARN-3049:
-
bq. Also, to Li Lu's point, we should provide an additional api
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660923#comment-14660923
]
Vrushali C commented on YARN-3049:
--
It looks like the conversion back to String was done t
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660902#comment-14660902
]
Li Lu commented on YARN-3049:
-
A little bit more investigation shows that we're using Strings f
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660877#comment-14660877
]
Li Lu commented on YARN-3049:
-
Hi [~vrushalic], I think the conversion to string happens on the
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660867#comment-14660867
]
Vrushali C commented on YARN-3049:
--
Hi [~zjshen]
In my experience, that kind of conversion
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660853#comment-14660853
]
Zhijie Shen commented on YARN-3049:
---
Here's a quick example:
{code}
@Test
public void
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660444#comment-14660444
]
Sangjin Lee commented on YARN-3049:
---
The latest patch (v.7) looks good to me.
Which time
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14659413#comment-14659413
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14659256#comment-14659256
]
Sangjin Lee commented on YARN-3049:
---
The latest patch looks good to me overall. Just a co
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14659012#comment-14659012
]
Li Lu commented on YARN-3049:
-
Hi [~zjshen], letting HBase implementation locally looks good to
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14654737#comment-14654737
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14654341#comment-14654341
]
Sangjin Lee commented on YARN-3049:
---
{quote}
I'm trying to understand the discussion here
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652854#comment-14652854
]
Sangjin Lee commented on YARN-3049:
---
{quote}
Then, we uniformly process the entities no m
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652850#comment-14652850
]
Li Lu commented on YARN-3049:
-
bq. What we discussed so far implies that we cannot only treat t
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652823#comment-14652823
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652806#comment-14652806
]
Zhijie Shen commented on YARN-3049:
---
Okay, what will the timestamp be used to do? If ther
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652709#comment-14652709
]
Sangjin Lee commented on YARN-3049:
---
I like that approach better than the previous. Thank
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652206#comment-14652206
]
Zhijie Shen commented on YARN-3049:
---
Hi Sangjin,
Thanks for your comments. The proposed
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14652140#comment-14652140
]
Sangjin Lee commented on YARN-3049:
---
When {{HBaseTimelineWriterImpl}} processes events fo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14650114#comment-14650114
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14650106#comment-14650106
]
Zhijie Shen commented on YARN-3049:
---
What I meant before is that HBaseTimelineWriterImpl
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14650075#comment-14650075
]
Sangjin Lee commented on YARN-3049:
---
I thought that the application created event would b
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14649915#comment-14649915
]
Zhijie Shen commented on YARN-3049:
---
I uploaded a new patch to address Sangjin's comments
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14649770#comment-14649770
]
Zhijie Shen commented on YARN-3049:
---
[~sjlee0], yeah, I agree it's not a decent solution
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14649658#comment-14649658
]
Sangjin Lee commented on YARN-3049:
---
Sorry [~zjshen] it took me a while to get to this. T
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14648336#comment-14648336
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14648243#comment-14648243
]
Li Lu commented on YARN-3049:
-
Hi [~zjshen]! Some of my comments:
bq. As I see a lot of argume
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14648218#comment-14648218
]
Zhijie Shen commented on YARN-3049:
---
[~gtCarrera9], thanks for review. I've addressed mos
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14646893#comment-14646893
]
Li Lu commented on YARN-3049:
-
Hi [~zjshen], some of my comments:
- The addition on {{newApp}}
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14646465#comment-14646465
]
Li Lu commented on YARN-3049:
-
Thanks [~zjshen]! For now I think it's fine to include the chang
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14646339#comment-14646339
]
Zhijie Shen commented on YARN-3049:
---
TestApplicationPriority.testApplicationPriorityAlloc
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14645444#comment-14645444
]
Hadoop QA commented on YARN-3049:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vo
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14645196#comment-14645196
]
Li Lu commented on YARN-3049:
-
Given the progress on YARN-3949, shall we focus back onto this J
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14634352#comment-14634352
]
Zhijie Shen commented on YARN-3049:
---
[~sjlee0], yeah, for POC purpose, I temporally do fl
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14634326#comment-14634326
]
Sangjin Lee commented on YARN-3049:
---
I do see that you're adding a call to {{BufferedMuta
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630867#comment-14630867
]
Varun Saxena commented on YARN-3049:
[~zjshen], should cluster ID be mandatory in REST
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14627349#comment-14627349
]
Li Lu commented on YARN-3049:
-
Hi [~zjshen], I have a concern similar to [~sjlee0]'s, on readin
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14627080#comment-14627080
]
Sangjin Lee commented on YARN-3049:
---
Thanks [~zjshen] for your WIP patch! I skimmed throu
[
https://issues.apache.org/jira/browse/YARN-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14619347#comment-14619347
]
Zhijie Shen commented on YARN-3049:
---
Updated the title accordingly to describe the scope
48 matches
Mail list logo