[ https://issues.apache.org/jira/browse/BOOKKEEPER-769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15192581#comment-15192581 ]
ASF GitHub Bot commented on BOOKKEEPER-769: ------------------------------------------- GitHub user robindh opened a pull request: https://github.com/apache/bookkeeper/pull/27 BOOKKEEPER-769: Remove the Hedwig Code - Remove code directories for Hedwig code - Remove code directories under compat - Remove Hedwig related documentation - Remove references to Hedwig code in pom files * There is an unrelated findbugs violation in BookieWatcher which is not related to this change * There were flaky tests that failed locally but passed when I reran them You can merge this pull request into a Git repository by running: $ git pull https://github.com/robindh/bookkeeper RemoveHedwig Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bookkeeper/pull/27.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 #27 ---- commit 45ad0460555ca62ca58dbf776fba377be6ef1d36 Author: Robin Dhamankar <robindh@robins-macbook-air.local> Date: 2016-03-13T23:54:31Z Remove the Hedwig Code ---- > [DISCUSS] Remove hedwig from source tree > ---------------------------------------- > > Key: BOOKKEEPER-769 > URL: https://issues.apache.org/jira/browse/BOOKKEEPER-769 > Project: Bookkeeper > Issue Type: Bug > Reporter: Ivan Kelly > Assignee: Matteo Merli > Fix For: 4.4.0 > > > As the license issue, isn't forcing the issue right now, I'd like to change > this to a general discussion of the future of hedwig. Almost all the comments > about the c++ client apply to hedwig as a whole. Hedwig has only had 5 real > changes in the last year, only one in 2014. For all intensive purposes, it is > inactive, unmaintained and unsupported. AFAIK, noone is using it in > production in anything close to the form of the apache codebase. > However, by having it in the codebase, at the same level of > bookkeeper-server, we are indicating to users that it is in fact supported > software and considered as important as bookkeeper-server. So, I would like > to propose that, after the 4.3.0 release, we remove hedwig from the codebase. > Obviously, it will still be available in the git/svn history, and in all > releases <= 4.3.0. > -The hedwig c++ client has not had any real code change since november 2012. > It is not built as part of the main build. It is not tested as part of the > main build. It is effectively unmaintained.- > -In addition to this, it contains m4 files whose license status is unclear.- > -I propose we simple remove it from trunk and branch-4.2. The license issue > is currently blocking the release of 4.2.3. [~jiannan], [~farrellee], you > were the last people to submit anything for the client. Are you using in > production? Any objection to its removal from the main code tree?- -- This message was sent by Atlassian JIRA (v6.3.4#6332)