Go ahead and file a JIRA, make sure to include sufficient information on issue and expected behavior though.
In the chat you did not mention that you are in the PPMC. Provide enough context, infra gets many questions all the time. On Tue, Feb 12, 2019 at 10:07 PM Vinoth Chandar <[email protected]> wrote: > I asked. I was redirected to mentors and have a jira filed. > Shall I go ahead and file a INFRA jira? > > > On Tue, Feb 12, 2019 at 6:52 PM Vinoth Chandar <[email protected]> wrote: > > > 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 > >> > > >> > > > > > > > > > >>> > >> >>> > > > >> > > >> > > > > > > > > > >>> > >> >>> > > >> > > >> > > > > > > > > > >>> > >> >>> > >> > > >> > > > > > > > > > >>> > >> >> > >> > > >> > > > > > > > > > >>> > >> > >> > > >> > > > > > > > > > >>> > > > >> > > >> > > > > > > > > > >>> > > >> > > >> > > > > > > > > > >>> > >> > > >> > > > > > > > > > >> > >> > > >> > > > > > > > > > > >> > > >> > > > > > > > > > >> > > >> > > > > > > > > >> > > >> > > > > > > > >> > > >> > > > > > > >> > > >> > > > > > >> > > >> > > > > >> > > >> > > > >> > > >> > > >> > > >> > >> > > > > >> > > > >> > > >> > > >
