Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 6D306200C24 for ; Thu, 23 Feb 2017 14:23:37 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 6BCD3160B62; Thu, 23 Feb 2017 13:23:37 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id D6BC6160B50 for ; Thu, 23 Feb 2017 14:23:36 +0100 (CET) Received: (qmail 11091 invoked by uid 500); 23 Feb 2017 13:23:35 -0000 Mailing-List: contact dev-help@kafka.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@kafka.apache.org Delivered-To: mailing list dev@kafka.apache.org Received: (qmail 11080 invoked by uid 99); 23 Feb 2017 13:23:34 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 Feb 2017 13:23:34 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 660D3DFD9E; Thu, 23 Feb 2017 13:23:34 +0000 (UTC) From: ijuma To: dev@kafka.apache.org Reply-To: dev@kafka.apache.org Message-ID: Subject: [GitHub] kafka pull request #2588: KAFKA-4788: Revert "KAFKA-4092: retention.bytes sh... Content-Type: text/plain Date: Thu, 23 Feb 2017 13:23:34 +0000 (UTC) archived-at: Thu, 23 Feb 2017 13:23:37 -0000 GitHub user ijuma opened a pull request: https://github.com/apache/kafka/pull/2588 KAFKA-4788: Revert "KAFKA-4092: retention.bytes should not be allowed to be less than segment.bytes" The intent is good, but it needs to take into account broker configs as well. See KAFKA-4788 for more details. This reverts commit 4ca5abe8ee7578f602fb7653cb8a09640607ea85. You can merge this pull request into a Git repository by running: $ git pull https://github.com/ijuma/kafka kafka-4788 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/2588.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 #2588 ---- commit 56c197a904dbf7549534f6589e062c61e17e47b5 Author: Ismael Juma Date: 2017-02-23T13:19:06Z Revert "KAFKA-4092: retention.bytes should not be allowed to be less than segment.bytes" The intent is good, but it needs to take into account broker configs as well. See KAFKA-4788 for more details. This reverts commit 4ca5abe8ee7578f602fb7653cb8a09640607ea85. ---- --- 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 infrastructure@apache.org or file a JIRA ticket with INFRA. ---