[GitHub] incubator-eagle issue #403: EAGLE-510: Refactor HDFS audit log parser

2016-08-30 Thread yonzhang
Github user yonzhang commented on the issue: https://github.com/apache/incubator-eagle/pull/403 sure, we can remove that 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 en

[GitHub] incubator-eagle issue #403: EAGLE-510: Refactor HDFS audit log parser

2016-08-30 Thread peterkim95
Github user peterkim95 commented on the issue: https://github.com/apache/incubator-eagle/pull/403 I see, can we can still refactor by removing the unused method parseUser --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-eagle issue #403: EAGLE-510: Refactor HDFS audit log parser

2016-08-30 Thread yonzhang
Github user yonzhang commented on the issue: https://github.com/apache/incubator-eagle/pull/403 We may not change the code to use regex matching as it is too slow. We have achieved more than 10 times performance gain by using string operations than using regex. This is critical i