Re: Unapproved releases

2019-01-29 Thread Justin Mclean
Hi, > Would it be possible to get an exception only for the latest one until we > make our first release (expected end of this quarter)? IMO you would need to ask VP legal for permission. My guess (but I could be wrong) is that they would say no and just suggest that you just make an official

Re: Unapproved releases

2019-01-29 Thread Subbu Subramaniam
I also noticed that the only way to delete releases from this page (that I could find) was to click on edit, remove the zip file, which then takes me to a page that has a delete button, and I click on delete. I noticed that there are probably 100+ releases in that page. Is there a quicker way

Re: Unapproved releases

2019-01-29 Thread Felix Cheung
IMO the last one is after incubation which is the biggest problem here. Other releases were from 2017. Is there a way this user can make do with commit tag, and build from source? Also, I’d also encourage discussions or requests with this user to be on dev@ - as the community can benefit from

Re: Unapproved releases

2019-01-29 Thread Subbu Subramaniam
Hi Felix, Thanks for quick response. The one from Dec was made for a specific user, so we need to check with them (at least) before removing. I will start that process right away. It was made for them to do some performance measurements on Pinot. The previous one was too old, and did not have

Re: Unapproved releases

2019-01-29 Thread Felix Cheung
Hi Subbu How are these releases made and what are they useful for? Generally, it’s not good to have unofficial unvoted releases, and having these visible links on GitHub can be confusing. To me the biggest concern is github doesn’t give you a way to note these are unofficial and unsupported

[GitHub] codecov-io commented on issue #3762: Documentation review on Pinot Overview

2019-01-29 Thread GitBox
codecov-io commented on issue #3762: Documentation review on Pinot Overview URL: https://github.com/apache/incubator-pinot/pull/3762#issuecomment-458766440 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3762?src=pr=h1) Report > Merging

Re: Unapproved releases

2019-01-29 Thread Subbu Subramaniam
Hi Felix/Kishore, Justin has pointed out that we (Pinot) have snapshot releases that are not through the apache web site, and in his opinion these need to be removed. He has suggested that we discuss with our mentors and put out an appropriate note in this month's podling report. IIRC these

[GitHub] jackjlli opened a new pull request #3762: Documentation review on Pinot Overview

2019-01-29 Thread GitBox
jackjlli opened a new pull request #3762: Documentation review on Pinot Overview URL: https://github.com/apache/incubator-pinot/pull/3762 Minor fix. This is an automated message from the Apache Git Service. To respond to the

Re: Unapproved releases

2019-01-29 Thread Justin Mclean
Hi, > Is it sufficient if we write that the snapshots in the links are unofficial > snapshots of Pinot, and that we are working on a formal apache release? No you are no allowed to make unofficial releases that are public in this way. IMO they need to be removed. It would be best to discuss

[GitHub] jackjlli merged pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
jackjlli merged pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758 This is an automated message from the Apache Git Service. To respond to the

Re: Unapproved releases

2019-01-29 Thread Subbu Subramaniam
Hi Justin, Sure, we can include something in the report. Is it sufficient if we write that the snapshots in the links are unofficial snapshots of Pinot, and that we are working on a formal apache release? (the latter part is there in the report anyway) thanks -Subbu

[GitHub] jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252033372 ## File path:

[GitHub] jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252033312 ## File path:

[GitHub] jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
jackjlli commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252033190 ## File path:

Unapproved releases

2019-01-29 Thread Justin Mclean
Hi, I'm a little concerned about the releases you have publicly available on GitHub [1] as per [2] you must not point outsiders to any unapproved releases. Can you include in your incubator report (due shortly) how you will address this issue. Thanks, Justin P.S Please CC me on any replies as

[GitHub] Jackie-Jiang merged pull request #3761: Add Documents for Index Techniques

2019-01-29 Thread GitBox
Jackie-Jiang merged pull request #3761: Add Documents for Index Techniques URL: https://github.com/apache/incubator-pinot/pull/3761 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252000446 ## File path:

[GitHub] Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252001484 ## File path:

[GitHub] Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config

2019-01-29 Thread GitBox
Jackie-Jiang commented on a change in pull request #3758: Better handle NPE from getting instance config URL: https://github.com/apache/incubator-pinot/pull/3758#discussion_r252001160 ## File path:

[GitHub] akshayrai merged pull request #3760: [TE] Clean-up the migration logs

2019-01-29 Thread GitBox
akshayrai merged pull request #3760: [TE] Clean-up the migration logs URL: https://github.com/apache/incubator-pinot/pull/3760 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] akshayrai opened a new pull request #3760: [TE] Clean-up the migration logs

2019-01-29 Thread GitBox
akshayrai opened a new pull request #3760: [TE] Clean-up the migration logs URL: https://github.com/apache/incubator-pinot/pull/3760 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] jenniferdai merged pull request #3751: Adding pluggable storage docs

2019-01-29 Thread GitBox
jenniferdai merged pull request #3751: Adding pluggable storage docs URL: https://github.com/apache/incubator-pinot/pull/3751 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] snleee commented on a change in pull request #3722: Update LICENSE and NOTICE files

2019-01-29 Thread GitBox
snleee commented on a change in pull request #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#discussion_r251943840 ## File path: LICENSE ## @@ -199,4 +198,29 @@ distributed under the License is distributed on an "AS IS"

[GitHub] mcvsubbu commented on issue #3717: kafka consume problem

2019-01-29 Thread GitBox
mcvsubbu commented on issue #3717: kafka consume problem URL: https://github.com/apache/incubator-pinot/issues/3717#issuecomment-458632932 @ly923976094 I assume you don't have any redundancy on your servers and therefore you cannot restart it. Note that in this case you may not be able

[GitHub] felixcheung commented on a change in pull request #3722: Update LICENSE and NOTICE files

2019-01-29 Thread GitBox
felixcheung commented on a change in pull request #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#discussion_r251725922 ## File path: LICENSE ## @@ -199,4 +198,29 @@ distributed under the License is distributed on an "AS