[GitHub] eagle issue #876: [IMPROVEMENT] Application base on Apache Beam
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/876 @jhsenjaliya I will do more work on it, so i closed the pr. ---
[GitHub] eagle pull request #876: [IMPROVEMENT] Application base on Apache Beam
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
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. ---
[GitHub] eagle issue #913: [EAGLE-1001] Compare Chart of JPM detail isn't sorted
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/913 @qingwen220 I create a jira https://issues.apache.org/jira/browse/EAGLE-1003 for GenericEntityService improvement. --- 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 #913: [EAGLE-1001] Compare Chart of JPM detail isn't sorted
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/913 @zombieJ For now as a workaround, I will sort joblist by startTime in js engine. @qingwen220 In the future,I think GenericEntityService should sort by timestamp by default like MRJobExecutionResource does. And GenericEntityService should support sort by custom defined field. Correct me if I'm wrong. --- 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...
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 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 #911: Add Java Object serial file to rat plugin exclusion
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/911 Add Java Object serial file to rat plugin exclusion Add Java Object serial file to rat plugin exclusion You can merge this pull request into a Git repository by running: $ git pull https://github.com/r7raul1984/eagle MINOR-rat Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/911.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 #911 commit afac6af70883ba8ef1e918bf77e25b53a3f7a0b9 Author: r7raul1984 Date: 2017-04-14T04:55:08Z Add Java Object serial file to rat plugin exclusion. --- 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...
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 #903: [EAGLE-986] HBaseStorage unable to deserialize Deserialize...
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/903 @qingwen220 @zombieJ pls help review --- 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 #902: [EAGLE-985] Upgrade hbase dependency to 1.2.5
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/902 In my opition ,we should do more test before upgrade to hbase 1.2.5 rather than just solve compile 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 issue #902: [EAGLE-985] Upgrade hbase dependency to 1.2.5
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/902 @wujinhu +1 --- 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
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 #903: [EAGLE-986] HBaseStorage unable to deserialize Dese...
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 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
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 pull request #891: [MINOR] Fix transform metric exception problem
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/891 [MINOR] Fix transform metric exception problem - Open transform function and use systemMetricDescriptor for sys metric. You can merge this pull request into a Git repository by running: $ git pull https://github.com/r7raul1984/eagle minor-counttorate Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/891.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 #891 commit 64921190dff77f38b53d576e43f719d897a67098 Author: r7raul1984 Date: 2017-03-23T02:15:44Z Open transform function and use systemMetricDescriptor for sys metric. --- 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
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 pull request #885: [MINOR] Modify some type error.
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/885 [MINOR] Modify some type error. You can merge this pull request into a Git repository by running: $ git pull https://github.com/1haodian/eagle minor Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/885.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 #885 commit 7903eaa5ad10c0722e1c97337cd32122a4a8d8b4 Author: r7raul1984 Date: 2017-03-17T07:53:12Z Modify some type error. --- 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 #882: EAGLE-963 list & detail page use same durationTime
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/882 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 #877: fix some hdfs dashbord problem
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
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...
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 issue #874: [MINOR] Fix query criteria in hdfs dashboard.
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/874 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 #876: [IMPROVEMENT] Application base on Apache Beam
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/876 [IMPROVEMENT] Application base on Apache Beam - Use beam pipeline powered by spark to run log parse application. Just for reviewing , don't merge. You can merge this pull request into a Git repository by running: $ git pull https://github.com/r7raul1984/eagle beam Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/876.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 #876 commit f31b671e88232d517ba99d802023799a7e43f1f6 Author: r7raul1984 Date: 2017-02-22T08:00:54Z Add beam application and enviroment . commit f6908d4f8cd591f8bb34f60813f9d7577881c599 Author: r7raul1984 Date: 2017-02-22T08:41:12Z Merge remote-tracking branch 'origin/master' into master-beam commit 9e31f5c8e946651035893e12bc62e931ccee69bf Author: r7raul1984 Date: 2017-02-23T02:59:12Z Beam app draft. commit d8f023daaa002268b0e30de263f2515000deda7e Author: r7raul1984 Date: 2017-02-23T05:03:39Z Merge remote-tracking branch 'origin/master' into master-beam commit 9529d3a8390d9563d8beb7f285e686b7c1e555d0 Author: r7raul1984 Date: 2017-02-23T08:22:56Z Add beam application module. commit 984eac4f94d6168e36d5e9da681368fc7840d334 Author: r7raul1984 Date: 2017-02-27T08:45:55Z Add beam application with kafka support . commit eb0995982e3c02cbcb411f7d1308266ff639562c Author: r7raul1984 Date: 2017-02-28T02:51:49Z Use sparkpipeline result to track app status. commit accc2b8a55a8596ad24e2bea6470d9031f2e00ed Author: r7raul1984 Date: 2017-02-28T03:05:47Z Modify metric system test to support kafka 0.9 api. commit bdec4a7c01684725b7f76e1580504ab80275b9ac Author: r7raul1984 Date: 2017-02-28T04:45:40Z Modify some code. commit 8c5e45c4b7fdb9f9ad12aac09025cd08147ada35 Author: r7raul1984 Date: 2017-03-10T05:51:11Z Add kafka8IO dependency. commit 3bf3c6bd3e74380ac60534c765f704ce040af245 Author: r7raul1984 Date: 2017-03-10T05:55:58Z Add beam application and enviroment . commit d4215a1dcca91ddf868e3759bea3fd143627d78a Author: r7raul1984 Date: 2017-03-10T06:35:06Z Fix some config commit 9c178a2994ce18da453d79c854f06aef1892c72f Author: r7raul1984 Date: 2017-03-10T09:09:20Z Fix some dependency and modify app code. commit 45a48292cee59bf4944c575ed93652165df018d2 Author: r7raul1984 Date: 2017-03-13T05:22:18Z Fix kafka serializationexception by cast timestamp to string type. commit 3d5c608e56efee795c83aed81930b995a7d186c4 Author: r7raul1984 Date: 2017-03-13T07:20:36Z Modify dependency version. --- 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...
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 #817: [MINOR] Revert index.html
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 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 #810: [EAGLE-898][EAGLE-899] Improve user experience for ...
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/810 [EAGLE-898][EAGLE-899] Improve user experience for HBase JMX Dashboard - EAGLE-898 Show jmx dashboard background first, then render the chart as that in JPM application - EAGLE-899 Show two charts in a line by default https://issues.apache.org/jira/browse/EAGLE-898 https://issues.apache.org/jira/browse/EAGLE-899 You can merge this pull request into a Git repository by running: $ git pull https://github.com/1haodian/eagle EAGLE-898-AND-EAGLE-899 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/810.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 #810 commit 84b29e901df80816a3fde10d0b5c371a74868c73 Author: r7raul1984 Date: 2017-02-16T08:39:12Z [EAGLE-898][EAGLE-899] Improve user experience for HBase JMX Dashboard - EAGLE-898 Show jmx dashboard background first, then render the chart as that in JPM application - EAGLE-899 Show two charts in a line by default https://issues.apache.org/jira/browse/EAGLE-898 https://issues.apache.org/jira/browse/EAGLE-899 --- 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 #800: [EAGLE-892] fixng NPE on ResourceFetcher
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/800 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 #797: [EAGLE-866] Refactor HBase JMX Metric with appropri...
Github user r7raul1984 commented on a diff in the pull request: https://github.com/apache/eagle/pull/797#discussion_r100252845 --- Diff: eagle-hadoop-metric/src/main/webapp/app/apps/hadoop_metric/index.js --- @@ -23,35 +23,35 @@ var hadoopMetricApp = register(['ngRoute', 'ngAnimate', 'ui.router', 'eagle.service']); hadoopMetricApp.route("HadoopMetric", { - url: "/hadoopMetric?startTime&endTime", + url: "/Service/HBase/overview?startTime&endTime", --- End diff -- I think "/Service/HBase/overview" should be "/Services/HBase/overview" --- 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...
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 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 issue #783: [EAGLE-872] Transform counter metric to rate metric
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/783 @haoch If metric name endswith "count" , it will be converted to rate. --- 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 #792: [EAGLE-868] Refactor metric chart render process to...
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 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 Date: 2017-01-24T02:44:10Z Fix option choose error bug. commit 794b1acbc1235d47872fff82e4d3bc57ecdea6e6 Author: r7raul1984 Date: 2017-01-24T02:57:14Z Remove sourceURL. commit 6bce14755a49958263a3bafb53b5572fef174642 Author: r7raul1984 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 issue #791: [EAGLE-868] Refactor metric chart render process to improv...
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: [EAGLE-868] Refactor metric chart render process to...
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 pull request #791: [EAGKE-868] Refactor metric chart render process to...
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 issue #787: [MINOR] Change default-rack to /default-rack
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 #787: [MINOR] Change default-rack to /default-rack
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 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 #784: EAGLE-859 MapReduce job performance suggestion
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/784 @qingwen220 Checkstyle 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 #783: [EAGLE-872] Transform counter metric to rate metric
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 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 #772: [EAGLE-860] TopologyDataExtractor can't extract rig...
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 pull request #772: [EAGLE-860] TopologyDataExtractor can't extract rig...
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 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 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 issue #769: [MINOR] Change AlertEagleStorePlugin in sql file
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/769 @qingwen220 I think AlertHbaseStorePlugin and AlertJDBCStorePlugin should be insert into the table at start time. User can choose jdbc and hbase from web UI. Also,user can config it from web UI. --- 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 #769: [MINOR] Change AlertEagleStorePlugin in sql file
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/769 @qingwen220 I suggest we should rename these two classname to AlertHbaseStorePlugin and AlertJDBCStorePlugin. For now,these name is confusing. --- 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
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 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 #765: [EAGLE-851] Add validation for policy name
Github user r7raul1984 commented on a diff in the pull request: https://github.com/apache/eagle/pull/765#discussion_r95098855 --- Diff: eagle-server/src/main/webapp/app/dev/public/js/ctrls/alertEditCtrl.js --- @@ -304,6 +304,15 @@ return false; }; + + $scope.checkPolicyName = function () { + if($scope.policy.name.length > 50) { + return "length should less than 50"; + } + return false; --- End diff -- Yes,I agree with you.But I see checkPublisherName return false. So I consistent with checkPublisherName method. --- 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 #765: [EAGLE-851] Add validation for policy name
Github user r7raul1984 commented on a diff in the pull request: https://github.com/apache/eagle/pull/765#discussion_r95098715 --- Diff: eagle-server/src/main/webapp/app/dev/public/js/ctrls/alertEditCtrl.js --- @@ -304,6 +304,15 @@ return false; }; + + $scope.checkPolicyName = function () { + if($scope.policy.name.length > 50) { + return "length should less than 50"; --- End diff -- This message will combine with "Name". Finally, user will see the message "Name length should less than 50". --- 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 #765: [EAGLE-851] Add validation for policy name
GitHub user r7raul1984 opened a pull request: https://github.com/apache/eagle/pull/765 [EAGLE-851] Add validation for policy name - Add checkPolicyName method to check the length of policy name. https://issues.apache.org/jira/browse/EAGLE-851 You can merge this pull request into a Git repository by running: $ git pull https://github.com/1haodian/eagle EAGLE-851 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/eagle/pull/765.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 #765 commit 965bb33747e14171833f9970aa1fecc603e37dd3 Author: r7raul1984 Date: 2017-01-09T02:38:21Z [EAGLE-851] Add validation for policy name - Add checkPolicyName method to check the length of policy name. https://issues.apache.org/jira/browse/EAGLE-851 --- 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 #758: [EAGLE-687] updating SiteEntityToRelation to accept null v...
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/758 @haoch 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. ---