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 3B2D9200C3D for ; Tue, 14 Mar 2017 23:11:57 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 39BAC160B7E; Tue, 14 Mar 2017 22:11:57 +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 83D23160B63 for ; Tue, 14 Mar 2017 23:11:56 +0100 (CET) Received: (qmail 910 invoked by uid 500); 14 Mar 2017 22:11:55 -0000 Mailing-List: contact dev-help@reef.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@reef.apache.org Delivered-To: mailing list dev@reef.apache.org Received: (qmail 892 invoked by uid 99); 14 Mar 2017 22:11:55 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Mar 2017 22:11:55 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id F2591C0856 for ; Tue, 14 Mar 2017 22:11:54 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 3.129 X-Spam-Level: *** X-Spam-Status: No, score=3.129 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, HTML_MESSAGE=2, KAM_NUMSUBJECT=0.5, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SORBS_SPAM=0.5, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd1-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id iMelnmUJ_g6d for ; Tue, 14 Mar 2017 22:11:54 +0000 (UTC) Received: from mail-vk0-f51.google.com (mail-vk0-f51.google.com [209.85.213.51]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTPS id B9A8D5FAF3 for ; Tue, 14 Mar 2017 22:11:53 +0000 (UTC) Received: by mail-vk0-f51.google.com with SMTP id x75so59394912vke.2 for ; Tue, 14 Mar 2017 15:11:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=xjlI5YvKONSyfg2/RdmWRTnnLOyJwQZVzicUp6iG5po=; b=pqPE3KlytloVqQb0t6aV8Gx+ymAFUp/aJahNGDHhAbDuOQ1Lyr9KDqLVeHNlSX3AIC bAV930fWfY9IXmpCtCAT2upp064XEFKuEJb6w4Yn1DLp5IZmgnBivnxFzzyG5l6xeWGC yy32lfSfJFqp4GLbtqGWMSBneMl8MP2tPOnkVlgC0anD6Cb+GC4AzNVh/lu+PhckRNkn sQ0aE0+WPkObHdniW03wCS+Ji/Ly4ZQIsrMf67t3MYgp1teezLP6mpfNGmdEdK8zcMVt Gc7LIDQ9aBPApfOg7mA/17uGgJQxwGafUlDZEeb1h9PI8zRFotwshUaxJUMO1ugDW9ri uWCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=xjlI5YvKONSyfg2/RdmWRTnnLOyJwQZVzicUp6iG5po=; b=DpTH2PYxLh4VLlKdA0ONMxul0Drbv4QRchtp0FhlwSJEyMDW7JtgiGdsGfybQy/U+b xMCXmm2osy+LwW89oAYMiR2gf9S/0MzQl9CS7OTtR6JxPCNBQN47WBUVLhl45HeogUI4 E5lC025WOXvjdRof5fFDOGOfCMoH5MjDgIkLiE6XsXi6YBWiHnK3+QmtX+UkjoAGfyVO vYAH/1ajxBSN0FnPRHIZvq6F/b5VTE17vByrsreACkScN3H6hGUzk8Tr7nfp1q1lBRZl s4ufD9ee9zSbtwoGFtBW2j/hitTeFd2DCrBqbEIVTS+1uoQOutmW1sWQmX4AR1uWe5Fn xfeg== X-Gm-Message-State: AMke39n8LWEYy/sfuOTaR34gy3gtBv2mU7r1dvCp2mBfrxVpeVLc1PX3+zrISeWfs1eJQJsNp0Ss3SvU8EXGsg== X-Received: by 10.31.3.203 with SMTP id f72mr18732247vki.108.1489523730015; Tue, 14 Mar 2017 13:35:30 -0700 (PDT) MIME-Version: 1.0 Received: by 10.103.125.140 with HTTP; Tue, 14 Mar 2017 13:35:29 -0700 (PDT) In-Reply-To: References: From: Saikat Kanjilal Date: Tue, 14 Mar 2017 13:35:29 -0700 Message-ID: Subject: Re: A plan for reef-1583 To: dev@reef.apache.org Content-Type: multipart/alternative; boundary=001a114280b0cfeda2054ab6c46e archived-at: Tue, 14 Mar 2017 22:11:57 -0000 --001a114280b0cfeda2054ab6c46e Content-Type: text/plain; charset=UTF-8 For committing/pushing to your own master branch it'd be great to put a hook into running the .Net tests in an automated fashion, I just need to see a mechanism that is automated and spits out a report for all unit tests and stops a committ even if its to your own master branch. On Tue, Mar 14, 2017 at 1:27 PM, Julia Wang (QIUHE) < Qiuhe.Wang@microsoft.com.invalid> wrote: > Hi Saikat, > > "One thing I would like to see is for the unit tests to get automatically > run before every check in within the csharp codebase" > > We have AppVeyor to run .Net tests for every push to a branch during code > review time. You can also run .Net tests on a command line with msbuild > command just like mvn clean install. What others are you looking for? > > Thanks, > Julia > > -----Original Message----- > From: Saikat Kanjilal [mailto:sxk1969@gmail.com] > Sent: Tuesday, March 14, 2017 1:02 PM > To: dev@reef.apache.org > Subject: A plan for reef-1583 > > Folks, > For the past few days I've been working on a custom implementation of an > in memory cache so that we are not forcefully needing to make the > transition to .net framework 451, here are our options at this point, I > wanted to finish the implementation and the unit tests for this before I > articulated a plan: > > > What needs to be done to moving to .net framework 451? > - Upgrade the Org.Apache.Reef.Network project to use 451 > - Figure out what other dependent projects need upgrading as well > - No need to use my custom cache, we can try to move forward with > Microsfot.Extensions.Memory.MemoryCache > - Do a deep analysis on the ramifications on making this upgrade across > all of the codebase, do we need to fix unit/integration tests as well > - Refactor the code to get past the portability analyzer warnings (yes > this will occure even after upgrading to 451) > > > What if we don't upgrade > - I've attached a pull request to REEF-1583, we can use my custom in > memory cache, it works as far as unit tests are concerned , will need to > figure out if we need additional integration tests > - If we go this route I will need to fix the other portability analyzer > warnings by not upgrading other parts of the code as well > > > How long to make this decision: Well it'll be a committer/community > decision , I'd ideally like to make this in the next few days (namely 2-3 > days) > > One thing I would like to see is for the unit tests to get automatically > run before every check in within the csharp codebase (the equivalent of > doing mvn clean install in java), I would love some ideas on how to make > this happen. > > > Thanks in advance for helping me with this. > --001a114280b0cfeda2054ab6c46e--