[
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)
