[GitHub] eagle pull request #792: [EAGLE-868] Refactor metric chart render process to...

2017-01-23 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/792

[EAGLE-868] Refactor metric chart render process to improve user experience

[EAGLE-868] Refactor metric chart render process to improve user experience
- show "data loading icon" for each chart one by one
- load metric data query for each chart
- reader metric data into metric chart when response returned and remove 
"data loading icon" one by one

https://issues.apache.org/jira/browse/EAGLE-868

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle EAGLE-868

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/792.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #792


commit 5c2b6f40c1d2a3282e34700d9d72b219c4ca61e5
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-24T02:39:36Z

[EAGLE-868] Refactor metric chart render process to improve user experience
 - show "data loading icon" for each chart one by one
 - load metric data query for each chart
 - reader metric data into metric chart when response returned and remove 
"data loading icon" one by one

https://issues.apache.org/jira/browse/EAGLE-868

commit 24064d154b7603180add04f43b3f7df119919514
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-24T02:44:10Z

Fix option choose error bug.

commit 794b1acbc1235d47872fff82e4d3bc57ecdea6e6
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-24T02:57:14Z

Remove sourceURL.

commit 6bce14755a49958263a3bafb53b5572fef174642
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-24T06:55:15Z

Complete hbase master(active and standby) overview page and fix some JSLint 
problem.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #791: [EAGLE-868] Refactor metric chart render process to...

2017-01-23 Thread r7raul1984
Github user r7raul1984 closed the pull request at:

https://github.com/apache/eagle/pull/791


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #791: [EAGLE-868] Refactor metric chart render process to improv...

2017-01-23 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/791
  
I will modify some typo in commit message,so I close the pr.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #791: [EAGKE-868] Refactor metric chart render process to...

2017-01-23 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/791

[EAGKE-868] Refactor metric chart render process to improve user experience

- show "data loading icon" for each chart one by one
- load metric data query for each chart
- reader metric data into metric chart when response returned and remove 
"data loading icon" one by one

https://issues.apache.org/jira/browse/EAGLE-868

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle EAGLE868

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/791.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #791






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #831: [EAGLE-918] Improve eagle auth to support @PermitAl...

2017-02-21 Thread r7raul1984
Github user r7raul1984 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/831#discussion_r102352634
  
--- Diff: 
eagle-server/src/main/java/org/apache/eagle/server/authentication/config/UserAccount.java
 ---
@@ -0,0 +1,61 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ * 
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.eagle.server.authentication.config;
+
+import org.apache.eagle.common.authentication.UserPrincipal;
+
+import java.util.List;
+
+public class UserAccount {
+private String username;
+private String password;
+private List roles;
+
+public UserAccount(String username, String password) {
+this.username = username;
+this.password = password;
+}
+
+public String getUsername() {
+return username;
+}
+
+public void setUsername(String username) {
+this.username = username;
+}
+
+public String getPassword() {
+return password;
+}
+
+public void setPassword(String password) {
+this.password = password;
--- End diff --

 In order to make it more safe,I think that we should encrypt  the 
password .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #798: [EAGLE-864] Make sure "HBase" nav active in sidebar...

2017-02-09 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/798

[EAGLE-864] Make sure "HBase" nav active in sidebar when visiting through 
URL

 - When active submenu  then it's parent menu will active also.

https://issues.apache.org/jira/browse/EAGLE-864


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle EAGLE-864

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/798.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #798


commit ac311d28faeaba268cd1a54c85ef5d015d99131f
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-02-09T08:24:39Z

[EAGLE-864] Make sure "HBase" nav active in sidebar when visiting hbase 
monitoring web through URL
 - When active submenu  then it's parent menu will active also.

https://issues.apache.org/jira/browse/EAGLE-864




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #817: [MINOR] Revert index.html

2017-02-17 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/817

[MINOR] Revert index.html



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle Revert-index

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/817.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #817


commit 55f0bfd2c0569287abbd7b562b81452483302973
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-02-17T08:25:34Z

Revert index.html




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #783: [EAGLE-872] Transform counter metric to rate metric

2017-01-18 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/783

[EAGLE-872]  Transform counter metric to rate metric

- Add transform bolt using counterToRateFunction in 
HadoopMetricMonitorApp's storm topology.

https://issues.apache.org/jira/browse/EAGLE-872


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle EAGLE-872

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/783.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #783


commit d886628fced8fa352a3032613349bbda6c74fd90
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-19T05:23:41Z

[EAGLE-872]  Transform counter metric to rate metric

- Add transform bolt using counterToRateFunction in 
HadoopMetricMonitorApp's storm topology.

https://issues.apache.org/jira/browse/EAGLE-872




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #787: [MINOR] Change default-rack to /default-rack

2017-01-19 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/787

[MINOR] Change default-rack to /default-rack

- Change default-rack to /default-rack

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle DEFAULT-RACK

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/787.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #787


commit a1379c7016a32817775d4879f62efd313bb79af1
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-20T00:00:06Z

[MINOR] Change default-rack to /default-rack




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #787: [MINOR] Change default-rack to /default-rack

2017-01-19 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/787
  
@qingwen220  Because ClusterNodeAPITopologyRackResolver use "/default-rack" 
.I think it will issue some problem when  you complete EAGLE-876.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #769: [MINOR] Change AlertEagleStorePlugin in sql file

2017-01-09 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/769

[MINOR]  Change AlertEagleStorePlugin in sql file

- Change to 
org.apache.eagle.alert.engine.publisher.impl.AlertEagleStorePlugin


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle MINOR-AlertStore

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/769.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #769


commit 2b3436bcda9170e7e8fb9864b15f586b89d78d43
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-09T23:56:56Z

[MINOR]  Change AlertEagleStorePlugin in sql file
- Change to 
org.apache.eagle.alert.engine.publisher.impl.AlertEagleStorePlugin




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #772: [EAGLE-860] TopologyDataExtractor can't extract rig...

2017-01-12 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/772

[EAGLE-860] TopologyDataExtractor can't extract right rack info

- Use clusternode API to get rack info.

https://issues.apache.org/jira/browse/EAGLE-860

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1haodian/eagle EAGLE-860

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/772.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #772


commit b2fa3af8308a92c3f9bbde36104a6b522574a1d9
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-12T08:30:58Z

[EAGLE-860]  TopologyDataExtractor can't extract right rack info

- Use clusternode API to get rack info.

https://issues.apache.org/jira/browse/EAGLE-860

commit e522c5ff26e3b980c42cff9b77bf3162a4ce303c
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-01-12T08:33:53Z

Revert ui-build.sh




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #772: [EAGLE-860] TopologyDataExtractor can't extract rig...

2017-01-13 Thread r7raul1984
Github user r7raul1984 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/772#discussion_r95956299
  
--- Diff: 
eagle-topology-check/eagle-topology-app/src/main/java/org/apache/eagle/topology/resolver/model/NodeInfo.java
 ---
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or 
implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ *
+ */
+
+package org.apache.eagle.topology.resolver.model;
+
+import org.codehaus.jackson.annotate.JsonIgnoreProperties;
--- End diff --

OK


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #783: [EAGLE-872] Transform counter metric to rate metric

2017-03-20 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/783
  
@haoch  Thanks. I will merge.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #877: fix some hdfs dashbord problem

2017-03-16 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/877
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #879: MINOR: change the rest api for testing resource manager HA

2017-03-16 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/879
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #878: [MINOR] Add analysis_jobs, analysis_email metadata into me...

2017-03-16 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/878
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #903: [EAGLE-986] HBaseStorage unable to deserialize Dese...

2017-04-04 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/903

[EAGLE-986] HBaseStorage unable to deserialize Deserialize from java …

[EAGLE-986] HBaseStorage unable to deserialize  from java object bytes

- recover allocatedMB type from int to long

https://issues.apache.org/jira/browse/EAGLE-986

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/r7raul1984/eagle EAGLE-986

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/903.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #903


commit 37050b01b14acd5713ab4aef024eabe9781d116d
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-04-05T02:42:44Z

[EAGLE-986] HBaseStorage unable to deserialize Deserialize from java object 
bytes

- recover allocatedMB type from int to long

https://issues.apache.org/jira/browse/EAGLE-986




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #898: EAGLE-988 update sortpath of job list

2017-04-01 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/898
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #903: [EAGLE-986] HBaseStorage unable to deserialize Deserialize...

2017-04-05 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/903
  
@zombieJ  OK


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #812: [EAGLE-906] org.apache.storm 1.x with API changes

2017-04-05 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/812
  
@jhsenjaliya  In my opition , I don't think it is good time to upgrade 
storm to 1.1.0.  In my experience ,many companies do not use the latest version 
.If you insist to upgrade storm, I recommend  we should support multi storm 
version (like hive shims moudle) and do more integration test instead of unit 
test by mock.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #913: [EAGLE-1001] Compare Chart of JPM detail isn't sort...

2017-04-14 Thread r7raul1984
GitHub user r7raul1984 opened a pull request:

https://github.com/apache/eagle/pull/913

[EAGLE-1001] Compare Chart of JPM detail isn't sorted

- sort job list by startTime 

https://issues.apache.org/jira/browse/EAGLE-1001

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/r7raul1984/eagle EAGLE-1001

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/913.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #913


commit 110750e63c51c28bde49e53583f377df61b3a7cf
Author: r7raul1984 <tangji...@yhd.com>
Date:   2017-04-14T07:06:12Z

[EAGLE-1001] Compare Chart of JPM detail isn't sorted in the horizontal axis

- sort job list by startTime

https://issues.apache.org/jira/browse/EAGLE-1001




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #876: [IMPROVEMENT] Application base on Apache Beam

2017-09-05 Thread r7raul1984
Github user r7raul1984 closed the pull request at:

https://github.com/apache/eagle/pull/876


---


[GitHub] eagle issue #876: [IMPROVEMENT] Application base on Apache Beam

2017-09-04 Thread r7raul1984
Github user r7raul1984 commented on the issue:

https://github.com/apache/eagle/pull/876
  
@jhsenjaliya no update for this PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---