Re: [FINAL DRAFT] Incubator PMC Board Report - June 2016 -- feedback

Thanks Julian! Good to know. Cheers, Jignesh > On Jun 7, 2016, at 11:17 AM, Julian Hyde wrote: > > Some background on formatting. The Board uses plain text for its agenda and > minutes. The Wiki is just a convenient tool that allows several podlings to > edit their reports, which are then c

Re: Where are design discussions happening?

And by the way, Harshad and Zuyu are JIRA administrators, so they could potentially do this too. > On Jun 7, 2016, at 10:53 AM, Julian Hyde wrote: > > Yes, happy to do that. Can you make sure that jign...@apache.org is a JIRA > user and tell me the JIRA id. > > >> On Jun 7, 2016, at 10:48 AM

Re: Where are design discussions happening?

Yes, happy to do that. Can you make sure that jign...@apache.org is a JIRA user and tell me the JIRA id. > On Jun 7, 2016, at 10:48 AM, Jignesh Patel wrote: > > Dear Julian, > > I believe that is not me. I tried to see if I created that account and forgot > the password, but it does not appe

Re: Where are design discussions happening?

Dear Julian, I believe that is not me. I tried to see if I created that account and forgot the password, but it does not appear that way. After a bit of chasing down, I think that email belongs to https://plus.google.com/108466782913065225351/posts

[GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo...

Github user zuyu commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 @pateljm FYI, the style guide does not specify the order of any input arguments, including `query_id`. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-quickstep pull request #8: Added query id in all the operators and...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-quickstep/pull/8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo...

Github user pateljm commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 @hbdeshmukh Ok -- I see your point that the style guide contradicts putting the query_id first. How about we close this PR, and have a separate discussion on changes we need to make to the

[GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo...

Github user zuyu commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 @hbdeshmukh Ordering the input arguments is not only a code style thing, but putting `query_id` the first is somewhat the logical top-down flow from a query-level info down to a operator-leve

Re: Where are design discussions happening?

Jignesh, It looks as if you have Administrator access to the QUICKSTEP component in JIRA. The user who is administrator is “jigneshmpatel" user (jignashmpatel at gmail dot com). So, you should be able to create issues (as, in fact should anyone with committer or contributor access) and you sho

Re: [FINAL DRAFT] Incubator PMC Board Report - June 2016 -- feedback

Some background on formatting. The Board uses plain text for its agenda and minutes. The Wiki is just a convenient tool that allows several podlings to edit their reports, which are then combined into an Incubator report and incorporated in the agenda. The minutes of the last meeting are online[

[GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo...

Github user hbdeshmukh commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 Hi @pateljm @zuyu I beg to differ on this point. In most places I tried to conform to the Google C++ style guide which we use as a reference. The relevant section in the style guide can

[GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo...

Github user pateljm commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 @hbdeshmukh: Zuyu (@zuyu) makes an excellent point, IMHO. The code would be more readable if we put the query_id first. This argument is somewhat inconsistently placed in the argument list

Re: [FINAL DRAFT] Incubator PMC Board Report - June 2016 -- feedback

Hi Julian and Marvin: Changes made to https://wiki.apache.org/incubator/June2016 . Thanks for the feedback! Good to know about not using wiki markup. The wiki edit page is confusing as it shows in the bottom pane the markup syntax. That seems to encou

Re: Where are design discussions happening?

Anyone know how to file a Jira for Quickstep? My Jira login is “jignesh” — I can’t find a “create button” Another question — there appears to be too many different logins. Not sure if I am doing something wrong, and not finding a way to have a unified way of managing my identity across git, Jir

Re: Where are design discussions happening?

Good point Julian. Will open Jiras. The discussion is scattered as the team is all over for the summer and the PRs have the discussion as the person starting a PR explains what is in the feature. Sent from my iPhone > On Jun 6, 2016, at 7:47 PM, Julian Hyde wrote: > > There seems to be quite