[GitHub] eagle issue #650: [EAGLE-782] Upgrade org.codehaus.jackson to com.fasterxml....

2016-12-29 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/650 @jhsenjaliya I updated web.xml and resolve conflicts, thanks! By the way I find some class will refer org.codehaus.jettison.json.JSONException and org.codehaus.jettison.json.JSONObject

[GitHub] eagle issue #650: [EAGLE-782] Upgrade org.codehaus.jackson to com.fasterxml....

2016-12-30 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/650 @haoch please re-review this patch. Since we will release the new version at Feb., it would be low risk to merge this patch now. --- If your project is set up for it, you can reply to

[GitHub] eagle issue #765: [EAGLE-851] Add validation for policy name

2017-01-09 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/765 +1 on @haoch. By the way, from the perspective of priority, backend validation would be solved at first. --- If your project is set up for it, you can reply to this email and have your reply

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

2017-01-12 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/769 @r7raul1984 could you please push again to make chekc 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

[GitHub] eagle issue #778: [MINOR] Refactor series aggregation

2017-01-16 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/778 @chitin , I think it's better to rename title to "Refactor Percentile Chart for Assign, Bulk Assign and BalancerCluster" --- If your project is set up for it, you can reply to this

[GitHub] eagle issue #892: [MINOR] Fix Topology Check system config thrown NullPointe...

2017-03-28 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/892 @r7raul1984, Please apply following rules for PRs from yhder 1. **For any bug fix, if there is no UT please reject PR.** 2. **For any new feature of java end, if there is no UT please

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

2017-04-04 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/903 @r7raul1984 please make sure **long** is compatible with **int**. --- 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

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

2017-04-04 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/903 @r7raul1984 told me that long is not compatible with int, so even we revert it to long, we will still get deserialization issue. @qingwen220 how does ebay deploy eagle? Did you clean

[GitHub] eagle issue #902: [EAGLE-985] Upgrade hbase dependency to 1.2.5

2017-04-07 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/902 hi @tedyu Is the new API you used supported in 1.0.x? --- 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

[GitHub] eagle issue #908: [EAGLE-998]Add eagle csd

2017-04-10 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/908 @haoch could you have a look at this? --- 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

[GitHub] eagle issue #908: [EAGLE-998]Add eagle csd

2017-04-10 Thread baibaichen
Github user baibaichen commented on the issue: https://github.com/apache/eagle/pull/908 @chitin please adding 1. a mvn profile for build CDH parcel 2. readme.md --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as