Yes. The LDAP seems to be correct based on gitbox. I will ask in "askinfra" and get back.
According to LDAP, you will have access to the following repositories: > - *hudi:* > > > - incubator-hudi <https://github.com/apache/incubator-hudi> > > > - *incubator:* > > > - incubator <https://github.com/apache/incubator> > > On Tue, Feb 12, 2019 at 5:23 PM Thomas Weise <[email protected]> wrote: > Before that you could also ask in the asfinfra slack channel, since the > LDAP group is probably correct: > > http://people.apache.org/phonebook.html?podling=hudi > > > > On Tue, Feb 12, 2019 at 5:20 PM Suneel Marthi <[email protected]> wrote: > > > Which means only the mentors can now commit changes? That's not good. > > Someone please file and infra ticket to fix that. > > > > On Tue, Feb 12, 2019 at 8:17 PM Thomas Weise <[email protected]> wrote: > > > > > Check what you see here: > > > > > > https://github.com/orgs/apache/teams?query=@vinothchandar > > > > > > I suspect you would need be member of: > > > > > > https://github.com/orgs/apache/teams/hudi-committers > > > > > > > > > > > > > > > > > > > > > On Tue, Feb 12, 2019 at 5:11 PM Thomas Weise <[email protected]> wrote: > > > > > > > I don't see you part of "apache/incubator-committers" - maybe that's > > the > > > > reason? > > > > > > > > > > > > On Tue, Feb 12, 2019 at 4:03 PM Vinoth Chandar <[email protected]> > > > wrote: > > > > > > > >> Hi Suneel, > > > >> > > > >> I have 2FA and 3 steps on gitbox all green. > > > >> > > > >> I have created one more PR to keep testing my perms > > > >> https://github.com/apache/incubator-hudi/pull/3 (Please don't merge > > > this) > > > >> > > > >> One thing is, I could n't find a team on GitHub for hudi. > > > >> > > > >> Can one of the mentors check out the team info on GitHub and see > what > > it > > > >> says? > > > >> > > > >> Thanks > > > >> Vinoth > > > >> > > > >> On Tue, Feb 12, 2019 at 12:04 PM Suneel Marthi < > > [email protected] > > > > > > > >> wrote: > > > >> > > > >> > Vinoth - see if this helps > > > >> > > > > >> > http://opennlp.apache.org/using-git.html > > > >> > > > > >> > On Tue, Feb 12, 2019 at 12:45 PM Vinoth Chandar < > [email protected]> > > > >> wrote: > > > >> > > > > >> > > I am working through gitbox setup. I think we all missed this. > > Once > > > I > > > >> get > > > >> > > it working, will respond with steps and later add it to docs.. > > > >> > > > > > >> > > On Mon, Feb 11, 2019 at 7:47 PM Thomas Weise <[email protected]> > > > wrote: > > > >> > > > > > >> > > > Vinoth, > > > >> > > > > > > >> > > > When your github account is properly linked, you would be able > > to > > > >> merge > > > >> > > the > > > >> > > > PR from the github UI and you also be assignable as reviewer. > > > >> > > > > > > >> > > > I would recommend to start documenting the setup for other > > folks. > > > >> That > > > >> > > > would fall under committer guidelines. > > > >> > > > > > > >> > > > Thomas > > > >> > > > > > > >> > > > > > > >> > > > > > > >> > > > > > > >> > > > On Mon, Feb 11, 2019 at 7:10 PM Suneel Marthi < > > > >> [email protected] > > > >> > > > > > >> > > > wrote: > > > >> > > > > > > >> > > > > I was able to merge it > > > >> > > > > > > > >> > > > > I think u need to logout/login from both github and gitbox > for > > > the > > > >> > > > changes > > > >> > > > > to take effect. > > > >> > > > > > > > >> > > > > On Mon, Feb 11, 2019 at 10:03 PM Vinoth Chandar < > > > >> [email protected]> > > > >> > > > wrote: > > > >> > > > > > > > >> > > > > > No. That does nt work either. > > > >> > > > > > > > > >> > > > > > @others , can you see perms to merge this pr? > > > >> > > > > > https://github.com/apache/incubator-hudi/pull/1 > > > >> > > > > > > > > >> > > > > > On Mon, Feb 11, 2019 at 6:56 PM Suneel Marthi < > > > >> > > [email protected] > > > >> > > > > > > > >> > > > > > wrote: > > > >> > > > > > > > > >> > > > > > > On Mon, Feb 11, 2019 at 9:33 PM Vinoth Chandar < > > > >> > [email protected]> > > > >> > > > > > wrote: > > > >> > > > > > > > > > >> > > > > > > > Nope :(. I have already added my apache email to > GitHub. > > > >> Thats > > > >> > > all > > > >> > > > it > > > >> > > > > > > > should take, right? > > > >> > > > > > > > > > > >> > > > > > > Not quite, u need to completely logout/login from github > > for > > > >> that > > > >> > > to > > > >> > > > be > > > >> > > > > > > functional > > > >> > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > https://github.com/apache/incubator-hudi/pull/1 > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > On Mon, Feb 11, 2019 at 6:23 PM Suneel Marthi < > > > >> > > > > [email protected] > > > >> > > > > > > > > > >> > > > > > > > wrote: > > > >> > > > > > > > > > > >> > > > > > > > > u shuld be able to merge PRs Vinoth? doesn't work > for > > > ya ? > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > On Mon, Feb 11, 2019 at 9:04 PM Vinoth Chandar < > > > >> > > > [email protected]> > > > >> > > > > > > > wrote: > > > >> > > > > > > > > > > > >> > > > > > > > > > Mentors, > > > >> > > > > > > > > > > > > >> > > > > > > > > > Should anyone in PPMC be able to merge PRs on > > GitHub? > > > I > > > >> > don't > > > >> > > > > have > > > >> > > > > > > > access > > > >> > > > > > > > > > to do that for this PR. > > > >> > > > > > > > > > > > > >> > > > > > > > > > Thanks > > > >> > > > > > > > > > Vinoth > > > >> > > > > > > > > > > > > >> > > > > > > > > > On Mon, Feb 11, 2019 at 12:01 PM Vinoth Chandar < > > > >> > > > > > > > > > [email protected]> wrote: > > > >> > > > > > > > > > > > > >> > > > > > > > > > > Bumpingn this thread again. > > > >> > > > > > > > > > > > > > >> > > > > > > > > > > Once we reciew and land the PR, it will be easy > to > > > >> make > > > >> > > > > > incremental > > > >> > > > > > > > > > > changes to reflect new guidelines and docs > tooling > > > >> > > > > > > > > > > > > > >> > > > > > > > > > > On Tue, Feb 5, 2019 at 8:49 AM Vinoth Chandar < > > > >> > > > > > > > > > > [email protected]> wrote: > > > >> > > > > > > > > > > > > > >> > > > > > > > > > >> Great points! Docker definitely could be useful > > to > > > >> > > provide a > > > >> > > > > > > > standard > > > >> > > > > > > > > > >> Jekyll, ruby build environment. > > > >> > > > > > > > > > >> Something, that Nishith himself had trouble > with. > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > >> On Mon, Feb 4, 2019 at 7:09 AM Thomas Weise < > > > >> > > [email protected] > > > >> > > > > > > > >> > > > > > > wrote: > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > >>> A pull request is the common way to propose > > > changes > > > >> in > > > >> > > > other > > > >> > > > > > > > > projects, > > > >> > > > > > > > > > so > > > >> > > > > > > > > > >>> that's good. > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> One of the things you will want to work on > going > > > >> > forward > > > >> > > > are > > > >> > > > > > > > > > contributor > > > >> > > > > > > > > > >>> guidelines that explain this, among other > > things. > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> Other suggestions (that can all be taken up as > > > >> > follow-up, > > > >> > > > > > perhaps > > > >> > > > > > > > > worth > > > >> > > > > > > > > > >>> creating a JIRAs): > > > >> > > > > > > > > > >>> - Source file should have license headers > > > >> > > > > > > > > > >>> - Take a look at other incubator and top level > > > >> projects > > > >> > > for > > > >> > > > > web > > > >> > > > > > > > site > > > >> > > > > > > > > > >>> commonalities such as community / contributing > > > >> sections > > > >> > > > > > > > > > >>> - Consider adding a README to the web site > > sources > > > >> > folder > > > >> > > > > and a > > > >> > > > > > > > build > > > >> > > > > > > > > > >>> script that shields developers from error > prone > > > >> manual > > > >> > > > steps > > > >> > > > > > > > > > >>> - Docker is a good way to avoid having to > > install > > > >> > pieces > > > >> > > on > > > >> > > > > the > > > >> > > > > > > > host > > > >> > > > > > > > > OS > > > >> > > > > > > > > > >>> (example: > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > https://flink.apache.org/improve-website.html#update-or-extend-the-documentation > > > >> > > > > > > > > > >>> ) > > > >> > > > > > > > > > >>> - Maybe in the future you want to separate web > > > site > > > >> > > sources > > > >> > > > > > from > > > >> > > > > > > > > > >>> documentation (since documentation is usually > > > >> specific > > > >> > to > > > >> > > > > > > versions) > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> Thomas > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> On Sun, Feb 3, 2019 at 11:05 PM nishith > agarwal > > < > > > >> > > > > > > > [email protected] > > > >> > > > > > > > > > > > > >> > > > > > > > > > >>> wrote: > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >>> > Hi All, > > > >> > > > > > > > > > >>> > > > > >> > > > > > > > > > >>> > I've altered the documentation and created a > > PR > > > >> > against > > > >> > > > the > > > >> > > > > > > > > > "asf-site" > > > >> > > > > > > > > > >>> > branch due to absence of a master branch > > > (instead > > > >> of > > > >> > > > > pushing > > > >> > > > > > > to a > > > >> > > > > > > > > > >>> branch > > > >> > > > > > > > > > >>> > directly). In the comments, I've tried to > > > explain > > > >> the > > > >> > > > > > changes I > > > >> > > > > > > > > made. > > > >> > > > > > > > > > >>> In > > > >> > > > > > > > > > >>> > summary, most of our packages, metadata for > > > >> datasets, > > > >> > > > admin > > > >> > > > > > > > client > > > >> > > > > > > > > > and > > > >> > > > > > > > > > >>> more > > > >> > > > > > > > > > >>> > are named with "hoodie". For now, I've just > > made > > > >> some > > > >> > > > > > cosmetic > > > >> > > > > > > > name > > > >> > > > > > > > > > >>> changes > > > >> > > > > > > > > > >>> > in the docs from "hoodie" -> "hudi" since > > > >> performing > > > >> > a > > > >> > > > > > > widespread > > > >> > > > > > > > > > >>> rename > > > >> > > > > > > > > > >>> > does not make sense. > > > >> > > > > > > > > > >>> > My guess is we can have an initial version > of > > > the > > > >> > site > > > >> > > > > ready > > > >> > > > > > > with > > > >> > > > > > > > > > such > > > >> > > > > > > > > > >>> > changes, iterate on it and come up with a > plan > > > as > > > >> we > > > >> > > > > migrate > > > >> > > > > > > the > > > >> > > > > > > > > > >>> codebase > > > >> > > > > > > > > > >>> > but I'd love to hear your thoughts around > > this. > > > >> > > > > > > > > > >>> > > > > >> > > > > > > > > > >>> > Thanks, > > > >> > > > > > > > > > >>> > Nishith > > > >> > > > > > > > > > >>> > > > > >> > > > > > > > > > >>> > On Thu, Jan 31, 2019 at 2:38 PM nishith > > agarwal > > > < > > > >> > > > > > > > > [email protected] > > > >> > > > > > > > > > > > > > >> > > > > > > > > > >>> > wrote: > > > >> > > > > > > > > > >>> > > > > >> > > > > > > > > > >>> > > Yes, I'll get to it later tonight. > > > >> > > > > > > > > > >>> > > > > > >> > > > > > > > > > >>> > > Thanks, > > > >> > > > > > > > > > >>> > > Nishith > > > >> > > > > > > > > > >>> > > > > > >> > > > > > > > > > >>> > > On Thu, Jan 31, 2019 at 1:04 PM Vinoth > > > Chandar < > > > >> > > > > > > > > [email protected]> > > > >> > > > > > > > > > >>> > wrote: > > > >> > > > > > > > > > >>> > > > > > >> > > > > > > > > > >>> > >> Nishith, are you working on the site? > Just > > > >> wanted > > > >> > to > > > >> > > > > > confirm > > > >> > > > > > > > and > > > >> > > > > > > > > > >>> see if > > > >> > > > > > > > > > >>> > >> you > > > >> > > > > > > > > > >>> > >> need more help from one of us. > > > >> > > > > > > > > > >>> > >> > > > >> > > > > > > > > > >>> > >> On Thu, Jan 24, 2019 at 8:21 PM Vinoth > > > Chandar > > > >> < > > > >> > > > > > > > > > >>> > >> [email protected]> wrote: > > > >> > > > > > > > > > >>> > >> > > > >> > > > > > > > > > >>> > >> > What thomas says, makes sense to me. > For > > > >> now, we > > > >> > > can > > > >> > > > > > just > > > >> > > > > > > > > import > > > >> > > > > > > > > > >>> site > > > >> > > > > > > > > > >>> > >> into > > > >> > > > > > > > > > >>> > >> > asf-site and make a call on what we > check > > > >> into > > > >> > > > master > > > >> > > > > > down > > > >> > > > > > > > the > > > >> > > > > > > > > > >>> line? > > > >> > > > > > > > > > >>> > >> > > > > >> > > > > > > > > > >>> > >> > On Thu, Jan 24, 2019 at 8:17 PM Vinoth > > > >> Chandar < > > > >> > > > > > > > > > >>> > >> > [email protected]> wrote: > > > >> > > > > > > > > > >>> > >> > > > > >> > > > > > > > > > >>> > >> >> Guess it does nt clutter source code > > with > > > >> docs, > > > >> > > js, > > > >> > > > > css > > > >> > > > > > > > etc? > > > >> > > > > > > > > > >>> > >> >> > > > >> > > > > > > > > > >>> > >> >> On Thu, Jan 24, 2019 at 7:46 PM > nishith > > > >> > agarwal < > > > >> > > > > > > > > > >>> [email protected] > > > >> > > > > > > > > > >>> > > > > > >> > > > > > > > > > >>> > >> >> wrote: > > > >> > > > > > > > > > >>> > >> >> > > > >> > > > > > > > > > >>> > >> >>> I vote for #3 as well. Yes, I'll take > > up > > > >> the > > > >> > > lead > > > >> > > > on > > > >> > > > > > > this. > > > >> > > > > > > > > > >>> > >> >>> BTW, why is creating a separate > branch > > > >> > > (asf-site) > > > >> > > > so > > > >> > > > > > > > > popular ? > > > >> > > > > > > > > > >>> I see > > > >> > > > > > > > > > >>> > >> that > > > >> > > > > > > > > > >>> > >> >>> many projects have done that. > > > >> > > > > > > > > > >>> > >> >>> > > > >> > > > > > > > > > >>> > >> >>> Thanks, > > > >> > > > > > > > > > >>> > >> >>> Nishith > > > >> > > > > > > > > > >>> > >> >>> > > > >> > > > > > > > > > >>> > >> >>> On Thu, Jan 24, 2019 at 2:13 PM Anbu > > > >> > Cheeralan < > > > >> > > > > > > > > > >>> [email protected]> > > > >> > > > > > > > > > >>> > >> >>> wrote: > > > >> > > > > > > > > > >>> > >> >>> > > > >> > > > > > > > > > >>> > >> >>> > I prefer #3 that will keep the > > > >> documentation > > > >> > > > > in-sync > > > >> > > > > > > > with > > > >> > > > > > > > > > the > > > >> > > > > > > > > > >>> > code. > > > >> > > > > > > > > > >>> > >> >>> > > > > >> > > > > > > > > > >>> > >> >>> > On 2019/01/24 21:27:39, Vinoth > > Chandar > > > < > > > >> > > > > > > > > > >>> > >> [email protected]> > > > >> > > > > > > > > > >>> > >> >>> > wrote: > > > >> > > > > > > > > > >>> > >> >>> > > Hello all, > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > Just summarizing how the current > > > >> project > > > >> > > > > > > documentation > > > >> > > > > > > > > is > > > >> > > > > > > > > > >>> setup > > > >> > > > > > > > > > >>> > >> and > > > >> > > > > > > > > > >>> > >> >>> then > > > >> > > > > > > > > > >>> > >> >>> > > use this thread to discuss how we > > > want > > > >> to > > > >> > > > > proceed. > > > >> > > > > > > Its > > > >> > > > > > > > > > >>> based on > > > >> > > > > > > > > > >>> > >> >>> Jekyll > > > >> > > > > > > > > > >>> > >> >>> > and > > > >> > > > > > > > > > >>> > >> >>> > > markup.To build Hudi docs, you > just > > > >> need > > > >> > to > > > >> > > > > > install > > > >> > > > > > > > > > >>> gem/ruby set > > > >> > > > > > > > > > >>> > >> and > > > >> > > > > > > > > > >>> > >> >>> spin > > > >> > > > > > > > > > >>> > >> >>> > > up Jekyll. > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > > > https://idratherbewriting.com/documentation-theme-jekyll/ > > > >> > > > > > > > > > >>> has > > > >> > > > > > > > > > >>> > >> >>> > > instructions. > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > We have a few options to manage > > docs. > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > 1. Separate repo (away from > source > > > >> code) > > > >> > > > > > > > > > >>> > >> >>> > > 2. Special branch along with code > > > >> > (asf-site > > > >> > > > > branch > > > >> > > > > > > > here > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> https://github.com/apache/incubator-hudi > > > >> > ) > > > >> > > > > > > > > > >>> > >> >>> > > 3. Checked into a top level docs > > > >> folder on > > > >> > > all > > > >> > > > > > > > branches > > > >> > > > > > > > > > >>> (current > > > >> > > > > > > > > > >>> > >> >>> > approach) > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > I vote for #3. I have do #2 on a > > > >> previous > > > >> > > open > > > >> > > > > > > source > > > >> > > > > > > > > > >>> project > > > >> > > > > > > > > > >>> > and > > > >> > > > > > > > > > >>> > >> the > > > >> > > > > > > > > > >>> > >> >>> > code > > > >> > > > > > > > > > >>> > >> >>> > > kept diverging from docs. > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > What do you all think? > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > Nishith, can you confirm you are > > > taking > > > >> > the > > > >> > > > lead > > > >> > > > > > on > > > >> > > > > > > > > this? > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > Thanks > > > >> > > > > > > > > > >>> > >> >>> > > Vinoth > > > >> > > > > > > > > > >>> > >> >>> > > > > > >> > > > > > > > > > >>> > >> >>> > > > > >> > > > > > > > > > >>> > >> >>> > > > >> > > > > > > > > > >>> > >> >> > > > >> > > > > > > > > > >>> > >> > > > >> > > > > > > > > > >>> > > > > > >> > > > > > > > > > >>> > > > > >> > > > > > > > > > >>> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > > > > >
