Return-Path: X-Original-To: apmail-aurora-dev-archive@minotaur.apache.org Delivered-To: apmail-aurora-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9E4FB1839E for ; Wed, 30 Dec 2015 00:19:17 +0000 (UTC) Received: (qmail 52798 invoked by uid 500); 30 Dec 2015 00:19:12 -0000 Delivered-To: apmail-aurora-dev-archive@aurora.apache.org Received: (qmail 52746 invoked by uid 500); 30 Dec 2015 00:19:12 -0000 Mailing-List: contact dev-help@aurora.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@aurora.apache.org Delivered-To: mailing list dev@aurora.apache.org Received: (qmail 52729 invoked by uid 99); 30 Dec 2015 00:19:12 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 30 Dec 2015 00:19:12 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id B831D180512 for ; Wed, 30 Dec 2015 00:19:11 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 3.653 X-Spam-Level: *** X-Spam-Status: No, score=3.653 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=3, SPF_NEUTRAL=0.652, URIBL_BLOCKED=0.001] autolearn=disabled Authentication-Results: spamd3-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=conductant-com.20150623.gappssmtp.com Received: from mx1-eu-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id 94sMTPnp1pmz for ; Wed, 30 Dec 2015 00:19:00 +0000 (UTC) Received: from mail-oi0-f54.google.com (mail-oi0-f54.google.com [209.85.218.54]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with ESMTPS id 05916203A3 for ; Wed, 30 Dec 2015 00:18:59 +0000 (UTC) Received: by mail-oi0-f54.google.com with SMTP id o124so195278979oia.1 for ; Tue, 29 Dec 2015 16:18:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=conductant-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=JRf48GqgACAiswuqEs52pc87iBoGIxGi0YSjgAjYUKU=; b=e0IzfTAk0FhZlUzov3BBQqzH8oTX8VkYcKNaMIZAM8fGDvDkEDQ7ohM/IxXMEeOLe5 3+qJ/uQS6pvgfx5R3QEChHpVYCAEA/+mlAFR2qI6Nu+q0Wgrbb57wT2h5babzE5nKDjq d/83Py9MbCJC8ak6KTRgdD9ace1bJqu3GdmqG1+gWzR9uR7z6s3t1AvpXkT3YXmIrDzT hfrlC1ghOEJOxXoPMKRtLV7oxumWK2uOXmTDUHm3KgHQe9RfSiD4j4gvifCwN2XZZnOl rM19j4yqeJ/QwBFIolE+jYaiQfavBJwGAu1Y6Is06artOg55ElP5439KRLANWzHRZU0X Y5WA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:content-type; bh=JRf48GqgACAiswuqEs52pc87iBoGIxGi0YSjgAjYUKU=; b=akKbh9uStRO9MVMm3WMDekB+Z5dQlR362OLwd0pDSbIguddL3FbUYfg9wMfGJsbhH3 w5rzMC6R4WRKrPQOG23qkabFBlSPe8JfKve54aJgdCLxAVSolEeMG2Xk6vozjOJ94e1Q OA9nV+7BIeOZPJrIE2xU/IWDRiJuAH3P0jgGwY0IrUhQtIiqECv7ZuaB816F75tFPYbW Y6dPKTMRuedqGSRWBe0njWGxklPxUJIH405nHYCDo15t+Pq43GHSAaarEYqWqrXQibWS XlM2m3aXZAgVZiQaHujKgqnv6FUrhlhK6OEoSzrxm5jAOyvPzvZ6mVct2lQyU7zCggvq fvlw== X-Gm-Message-State: ALoCoQmsxNZ+dzsPb/XyF65FuAJVkh+9SkMv/JbZWf2XN+NqSXuvUgKrVczaNczAUMQI7AwZwOQ5kCPPCQHGeB+wMM0sK+/yQA== MIME-Version: 1.0 X-Received: by 10.202.94.10 with SMTP id s10mr33356825oib.99.1451434733297; Tue, 29 Dec 2015 16:18:53 -0800 (PST) Received: by 10.60.118.71 with HTTP; Tue, 29 Dec 2015 16:18:53 -0800 (PST) X-Originating-IP: [166.173.63.144] In-Reply-To: References: Date: Tue, 29 Dec 2015 17:18:53 -0700 Message-ID: Subject: Re: [PROPOSAL] Use standard logging practices From: John Sirois To: dev@aurora.apache.org Content-Type: multipart/alternative; boundary=001a113d5cd2b1789b0528127b92 --001a113d5cd2b1789b0528127b92 Content-Type: text/plain; charset=UTF-8 On Tue, Dec 29, 2015 at 5:05 PM, John Sirois wrote: > > > On Tue, Dec 29, 2015 at 5:02 PM, Jeff Schroeder < > jeffschroeder@computer.org> wrote: > >> Primarily it is faster, uses less memory, and annotates tracebacks with >> package versions. The last one seems like a winner for debugging user >> issues or operationally. >> >> http://logback.qos.ch/reasonsToSwitch.html >> >> I'm not strongly opinionated either way, but it does seem like a better >> log4j. >> > > Looks like this decision is nicely limited to a build.gradle edit: > http://logback.qos.ch/reasonsToSwitch.html#slf4j > After a brief skim of the configuration docs [1], I'm in favor of switching in a follow-up RB to https://reviews.apache.org/r/41777/ In short - logback supports pointing to a non-root config file via a system-property out of the box, this makes aurora a non-nuisance for operators, they can easily modify init scripts to point to a custom config. [1] http://logback.qos.ch/manual/configuration.html > >> On Tuesday, December 29, 2015, Bill Farner wrote: >> >> > I don't have a strong opinion about logback vs log4j. Can you summarize >> > some of the tradeoffs? >> > >> > On Tue, Dec 29, 2015 at 3:52 PM, Jeff Schroeder < >> > jeffschroeder@computer.org > >> > wrote: >> > >> > > What about using logback instead of log4j? It has some interesting >> > benefits >> > > over log4j and we wouldn't be the first large mesos framework to >> switch >> > to >> > > it. >> > > >> > > Personally, I'd love to see glog burn and die in a fire. >> > > >> > > On Monday, December 28, 2015, Bill Farner > > > wrote: >> > > >> > > > We're currently using some logging scaffolding carried over from >> > Twitter >> > > > commons. I would like to propose that we dismantle some of this in >> > favor >> > > > of more standard java application logging conventions. >> > > > >> > > > Concretely, i propose we remove the following scheduler command line >> > > > arguments: >> > > > -logtostderr >> > > > -alsologtostderr >> > > > -vlog >> > > > -vmodule >> > > > -use_glog_formatter >> > > > >> > > > Instead of these, we can allow users to customize logging via >> standard >> > > > java.util.logging inputs (e.g. logging.properties). We could >> explore >> > > using >> > > > an alternative to java.util.logging, but i suggest we retain that >> > backend >> > > > for now (since it's what we're currently using). >> > > > >> > > >> > > >> > > -- >> > > Text by Jeff, typos by iPhone >> > > >> > >> >> >> -- >> Text by Jeff, typos by iPhone >> > > > > -- > John Sirois > 303-512-3301 > -- John Sirois 303-512-3301 --001a113d5cd2b1789b0528127b92--