Mailing-List: contact dev-help@mxnet.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@mxnet.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <CAMtqZefa36tcWmv+a9bWmvbktTrPFV6ZgHK-2F7X7grXOZ+XZw@mail.gmail.com>
References: <CAMtqZefa36tcWmv+a9bWmvbktTrPFV6ZgHK-2F7X7grXOZ+XZw@mail.gmail.com>
From: CodingCat <codingcat@apache.org>
Date: Sat, 2 Dec 2017 10:04:47 -0800
Message-ID: <CAMtqZeciTYsOK2oT_wyPbpxAj4opDFZYAECZrAYe6JwCFn+jUA@mail.gmail.com>
Subject: Re: request for reviewing PR in ps-lite
To: dev@mxnet.incubator.apache.org
Content-Type: multipart/alternative; boundary="f403045c53342895a9055f5f520f"
archived-at: Sat, 02 Dec 2017 18:04:56 -0000

--f403045c53342895a9055f5f520f
Content-Type: text/plain; charset="UTF-8"

ping

On Fri, Dec 1, 2017 at 12:18 AM, Nan Zhu <zhunanmcgill@gmail.com> wrote:

> Hi, all
>
> I have been working on integrating MXNet with Spark in a more full-fledged
> manner.
>
> One of the most critical pre-conditions is to make parameter server in
> mxnet support multiple workers per process. I created the PR in
> https://github.com/dmlc/ps-lite/pull/121 (OK, sorry for being late....I
> should have finished it earlier)
>
> This PR includes some refactoring of those too long methods, to highlight
> the changes
>
> 1. https://github.com/dmlc/ps-lite/pull/112 includes the changes related
> to refactoring
>
> 2. https://github.com/CodingCat/ps-lite/pull/3/files includes the changes
> related to the key functionality
>
> 3. https://github.com/dmlc/ps-lite/pull/121 contains everything (Please
> review this one)
>
>
> I am not sure who is the current owner of ps-lite, please help to share
> your thoughts on the implementation. Only after this PR is merged and
> ps-lite version is synced in mxnet repo, I can file the successive PRs in
> mxnet
>
> Thank you very much!
>
> Nan
>

--f403045c53342895a9055f5f520f--