Re: [DISCUSS] Graduate Apache Pinot (Incubating) as a TLP

2021-06-18 Thread Justin Mclean
Hi,

One other thing you might want to correct, all of the download links to 
previous version are broken on your download page. [1] The previous version are 
automatically archived and can be found here. [2] You may also want to consider 
not calling the convenience binary an “offical binary release”.

Thanks,
Justin

1. https://pinot.apache.org/download
2. https://archive.apache.org/dist/incubator/pinot/
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [DISCUSS] Graduate Apache Pinot (Incubating) as a TLP

2021-06-18 Thread Mayank Shrivastava
Thank you Felix for pointing this out, much appreciated.

Looking forward to guidance on the next steps in our journey to graduate
Pinot to TLP.


Best Regards,
Mayank

On Fri, Jun 18, 2021 at 1:25 PM Felix Cheung  wrote:

> To add, the thirdeye code directory was excluded from past releases and so
> was _not_ part of the Pinot project release. (A topic discussed several
> times on list)
>
>
> On Thu, Jun 17, 2021 at 4:54 PM Mayank Shrivastava 
> wrote:
>
> > Hi Justin,
> > Thank you for bringing up the discussion aspect. It seems that it may
> > purely have been an accidental oversight, but PPMCs were aware of the
> > discussions as well as the decision.The key reason was that the ThirdEye
> > project did not gain much traction in the open source, and as stated in
> [1]
> > the ThirdEye team felt running it under ASF did not have much return on
> > investment. Hope this helps clarify.
> >
> > [1] Discussion to move ThirdEye of Apache Pinot
> > <
> >
> https://lists.apache.org/thread.html/r23c350271bd907718c6b777b0091e493f93ab407443528d4d549172a%40%3Cprivate.pinot.apache.org%3E
> > >
> > Best Regards,
> > Mayank
> >
> > On Thu, Jun 17, 2021 at 4:09 PM Justin Mclean 
> > wrote:
> >
> > > Hi,
> > >
> > > Thanks for taking action on this.
> > >
> > > Still one things concerns me, the discussion of removal of ThirdEye is
> > > unusual as it happened offlist and it seems it was only brought back to
> > the
> > > list after a decision was made. For decisions like this, all of the PMC
> > > need to be informed and the decision needs to be made in the open on
> the
> > > list. The source of the issue to me seems to be commit rights and I
> > think a
> > > better way of resolving this would have been considering lowering the
> > > commit bar. Was that considered in the off list discussions?
> > >
> > > Kind Regards,
> > > Justin
> > >
> > >
> > > -
> > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > > For additional commands, e-mail: general-h...@incubator.apache.org
> > >
> > >
> >
>


Re: [DISCUSS] Graduate Apache Pinot (Incubating) as a TLP

2021-06-18 Thread Felix Cheung
To add, the thirdeye code directory was excluded from past releases and so
was _not_ part of the Pinot project release. (A topic discussed several
times on list)


On Thu, Jun 17, 2021 at 4:54 PM Mayank Shrivastava 
wrote:

> Hi Justin,
> Thank you for bringing up the discussion aspect. It seems that it may
> purely have been an accidental oversight, but PPMCs were aware of the
> discussions as well as the decision.The key reason was that the ThirdEye
> project did not gain much traction in the open source, and as stated in [1]
> the ThirdEye team felt running it under ASF did not have much return on
> investment. Hope this helps clarify.
>
> [1] Discussion to move ThirdEye of Apache Pinot
> <
> https://lists.apache.org/thread.html/r23c350271bd907718c6b777b0091e493f93ab407443528d4d549172a%40%3Cprivate.pinot.apache.org%3E
> >
> Best Regards,
> Mayank
>
> On Thu, Jun 17, 2021 at 4:09 PM Justin Mclean 
> wrote:
>
> > Hi,
> >
> > Thanks for taking action on this.
> >
> > Still one things concerns me, the discussion of removal of ThirdEye is
> > unusual as it happened offlist and it seems it was only brought back to
> the
> > list after a decision was made. For decisions like this, all of the PMC
> > need to be informed and the decision needs to be made in the open on the
> > list. The source of the issue to me seems to be commit rights and I
> think a
> > better way of resolving this would have been considering lowering the
> > commit bar. Was that considered in the off list discussions?
> >
> > Kind Regards,
> > Justin
> >
> >
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>


Re: [VOTE] Release Apache Flagon UserALE.js (Incubating) 2.2.0 (RC3)

2021-06-18 Thread John D. Ament
On Thu, Jun 17, 2021 at 9:52 PM Xun Liu  wrote:

> Hi, Johnson
>
> I have checked the following items:
>
> - Incubating in name
> - NOTICE is fine
> - DISCLAIMER exists
> - All links are valid
> - No unexpected JS files
>
> BUT, This file does not have ASF headers, Please fixed it.
>
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package-lock.json
>
> https://github.com/apache/incubator-flagon-useralejs/blob/master/cypress.json
>
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package.json


Please note that if you're going to review a release, you should be looking
at the release contents staged on the dist.apache.org URLs, not the current
contents on github (since we assume that the code may have already changed
for the new development cycle etc).


>
>
> Xun Liu
> Best regards
>
> On Thu, Jun 17, 2021 at 2:42 PM Gedd Johnson 
> wrote:
>
> > Hi Folks,
> >
> > Please VOTE on the Apache Flagon UserALE.js 2.2.0 Release Candidate #3.
> >
> > A VOTE was carried out on dev@flagon with the following results:
> >
> > [ 4 ] +1, let's get it released!!! (+2 (IPMC/Binding) +2 PPMC)
> > [ 0 ] +/-0, fine, but consider to fix few issues before...
> > [ 0 ] -1, nope, because... (and please explain why)
> >
> > We require +1 IPMC/Binding VOTE to Release.
> >
> > The VOTE thread can be found here:
> >
> >
> https://lists.apache.org/thread.html/rfa74ac6182a195facddecb4218f12b1354c02a0eee4bbb24d8d4d024%40%3Cdev.flagon.apache.org%3E
> >  (note
> > that the thread got a little convoluted, this link reflects the thread
> for
> > 2.2.0 RC3)
> >
> > About Flagon: http://flagon.incubator.apache.org/
> >
> > This Minor release includes:
> >
> > 1. Adding a page load log feature
> > 2. Refactoring the build pipeline using Rollup
> > 3. Adding a journey testing suite
> > 4. Fixing a bug with sendOnClose
> >
> > We solved 21 issues:
> > https://github.com/apache/incubator-flagon-useralejs/projects/7
> >
> > Git source tag (2b996df):
> >
> https://github.com/apache/incubator-flagon-useralejs/releases/tag/2.2.0-RC3
> >
> > Staging repo: https://dist.apache.org/repos/dist/dev/incubator/flagon/
> >
> > Source Release Artifacts:
> >
> >
> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.2.0-RC3/
> >
> > PGP release keys (signed using A88C905BD4516D4C):
> > https://dist.apache.org/repos/dist/dev/incubator/flagon/KEYS
> >
> > Link to successful Github CI Build:
> > https://github.com/apache/incubator-flagon-useralejs/actions
> >
> > Reference the useralejs testing framework to assist in your unit and
> > integration tests:
> >
> >
> https://cwiki.apache.org/confluence/display/FLAGON/UserALE.js+Testing+Framework
> >
> > Vote will be open for 72 hours. Please VOTE as follows:
> >
> > [ ] +1, let's get it released!!!
> > [ ] +/-0, fine, but consider to fix few issues before...
> > [ ] -1, nope, because... (and please explain why)
> >
> > Along with your VOTE, please indicate testing and checks you've made
> > against build artifacts, src, and documentation:
> >
> > [ ] Build and Unit Tests Pass
> > [ ] Integration Tests Pass
> > [ ] "Incubating" in References to Project and Distribution File Names
> > [ ] Signatures and Hashes Match Keys
> > [ ] DISCLAIMER, LICENSE, and NOTICE Files in Source and Binary Release
> > Packages
> > [ ] DISCLAIMER, LICENSE, and NOTICE are consistent with ASF and Incubator
> > Policy
> > [ ] CHANGELOG included with release distribution
> > [ ] All Source Files Have Correct ASF Headers
> > [ ] No Binary Files in Source Release Packages
> >
> > Thank you to everyone that is able to VOTE as well as everyone that
> > contributed to Apache Flagon UserALE.js (Incubating) 2.2.0
> >
>


Re: [VOTE] Release Apache Flagon UserALE.js (Incubating) 2.2.0 (RC3)

2021-06-18 Thread John D. Ament
+1 from me

On Thu, Jun 17, 2021 at 2:42 AM Gedd Johnson  wrote:

> Hi Folks,
>
> Please VOTE on the Apache Flagon UserALE.js 2.2.0 Release Candidate #3.
>
> A VOTE was carried out on dev@flagon with the following results:
>
> [ 4 ] +1, let's get it released!!! (+2 (IPMC/Binding) +2 PPMC)
> [ 0 ] +/-0, fine, but consider to fix few issues before...
> [ 0 ] -1, nope, because... (and please explain why)
>
> We require +1 IPMC/Binding VOTE to Release.
>
> The VOTE thread can be found here:
>
> https://lists.apache.org/thread.html/rfa74ac6182a195facddecb4218f12b1354c02a0eee4bbb24d8d4d024%40%3Cdev.flagon.apache.org%3E
>  (note
> that the thread got a little convoluted, this link reflects the thread for
> 2.2.0 RC3)
>
> About Flagon: http://flagon.incubator.apache.org/
>
> This Minor release includes:
>
> 1. Adding a page load log feature
> 2. Refactoring the build pipeline using Rollup
> 3. Adding a journey testing suite
> 4. Fixing a bug with sendOnClose
>
> We solved 21 issues:
> https://github.com/apache/incubator-flagon-useralejs/projects/7
>
> Git source tag (2b996df):
> https://github.com/apache/incubator-flagon-useralejs/releases/tag/2.2.0-RC3
>
> Staging repo: https://dist.apache.org/repos/dist/dev/incubator/flagon/
>
> Source Release Artifacts:
>
> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.2.0-RC3/
>
> PGP release keys (signed using A88C905BD4516D4C):
> https://dist.apache.org/repos/dist/dev/incubator/flagon/KEYS
>
> Link to successful Github CI Build:
> https://github.com/apache/incubator-flagon-useralejs/actions
>
> Reference the useralejs testing framework to assist in your unit and
> integration tests:
>
> https://cwiki.apache.org/confluence/display/FLAGON/UserALE.js+Testing+Framework
>
> Vote will be open for 72 hours. Please VOTE as follows:
>
> [ ] +1, let's get it released!!!
> [ ] +/-0, fine, but consider to fix few issues before...
> [ ] -1, nope, because... (and please explain why)
>
> Along with your VOTE, please indicate testing and checks you've made
> against build artifacts, src, and documentation:
>
> [ ] Build and Unit Tests Pass
> [ ] Integration Tests Pass
> [ ] "Incubating" in References to Project and Distribution File Names
> [ ] Signatures and Hashes Match Keys
> [ ] DISCLAIMER, LICENSE, and NOTICE Files in Source and Binary Release
> Packages
> [ ] DISCLAIMER, LICENSE, and NOTICE are consistent with ASF and Incubator
> Policy
> [ ] CHANGELOG included with release distribution
> [ ] All Source Files Have Correct ASF Headers
> [ ] No Binary Files in Source Release Packages
>
> Thank you to everyone that is able to VOTE as well as everyone that
> contributed to Apache Flagon UserALE.js (Incubating) 2.2.0
>


Re: [VOTE] Release Apache Flagon UserALE.js (Incubating) 2.2.0 (RC3)

2021-06-18 Thread Xun Liu
hi, Johnson

> Thanks for taking a look into this. The files mentioned are in JSON format
> which does not allow for comments. As far as we know, the ASF header is
not
> able to be included in this file type. See the linked example from the
> package.json file in Apache Airflow
> .

Thank you for reply :-)
+1 (non-binding) from me

Xun Liu
Best Regards

On Fri, Jun 18, 2021 at 2:11 PM Furkan KAMACI 
wrote:

> Hi,
>
> +1 from me (binding, carrying over my vote).
>
> I checked:
>
> - Incubating in name
> - DISCLAIMER exists
> - LICENSE and NOTICE are fine
> - No unexpected binary files
> - Checked PGP signatures
> - Checked checksums
> - Code compiles and tests successfully run
>
> Kind Regards,
> Furkan KAMACI
>
> On Fri, Jun 18, 2021 at 8:59 AM Gedd Johnson 
> wrote:
>
> > Hello Liu!
> >
> > Thanks for taking a look into this. The files mentioned are in JSON
> format
> > which does not allow for comments. As far as we know, the ASF header is
> not
> > able to be included in this file type. See the linked example from the
> > package.json file in Apache Airflow
> > .
> >
> > Best,
> > Gedd Johnson
> >
> > On Fri, Jun 18, 2021 at 3:52 AM Xun Liu  wrote:
> >
> > > Hi, Johnson
> > >
> > > I have checked the following items:
> > >
> > > - Incubating in name
> > > - NOTICE is fine
> > > - DISCLAIMER exists
> > > - All links are valid
> > > - No unexpected JS files
> > >
> > > BUT, This file does not have ASF headers, Please fixed it.
> > >
> > >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package-lock.json
> > >
> > >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/cypress.json
> > >
> > >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package.json
> > >
> > > Xun Liu
> > > Best regards
> > >
> > > On Thu, Jun 17, 2021 at 2:42 PM Gedd Johnson 
> > > wrote:
> > >
> > > > Hi Folks,
> > > >
> > > > Please VOTE on the Apache Flagon UserALE.js 2.2.0 Release Candidate
> #3.
> > > >
> > > > A VOTE was carried out on dev@flagon with the following results:
> > > >
> > > > [ 4 ] +1, let's get it released!!! (+2 (IPMC/Binding) +2 PPMC)
> > > > [ 0 ] +/-0, fine, but consider to fix few issues before...
> > > > [ 0 ] -1, nope, because... (and please explain why)
> > > >
> > > > We require +1 IPMC/Binding VOTE to Release.
> > > >
> > > > The VOTE thread can be found here:
> > > >
> > > >
> > >
> >
> https://lists.apache.org/thread.html/rfa74ac6182a195facddecb4218f12b1354c02a0eee4bbb24d8d4d024%40%3Cdev.flagon.apache.org%3E
> > > >  (note
> > > > that the thread got a little convoluted, this link reflects the
> thread
> > > for
> > > > 2.2.0 RC3)
> > > >
> > > > About Flagon: http://flagon.incubator.apache.org/
> > > >
> > > > This Minor release includes:
> > > >
> > > > 1. Adding a page load log feature
> > > > 2. Refactoring the build pipeline using Rollup
> > > > 3. Adding a journey testing suite
> > > > 4. Fixing a bug with sendOnClose
> > > >
> > > > We solved 21 issues:
> > > > https://github.com/apache/incubator-flagon-useralejs/projects/7
> > > >
> > > > Git source tag (2b996df):
> > > >
> > >
> >
> https://github.com/apache/incubator-flagon-useralejs/releases/tag/2.2.0-RC3
> > > >
> > > > Staging repo:
> https://dist.apache.org/repos/dist/dev/incubator/flagon/
> > > >
> > > > Source Release Artifacts:
> > > >
> > > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.2.0-RC3/
> > > >
> > > > PGP release keys (signed using A88C905BD4516D4C):
> > > > https://dist.apache.org/repos/dist/dev/incubator/flagon/KEYS
> > > >
> > > > Link to successful Github CI Build:
> > > > https://github.com/apache/incubator-flagon-useralejs/actions
> > > >
> > > > Reference the useralejs testing framework to assist in your unit and
> > > > integration tests:
> > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLAGON/UserALE.js+Testing+Framework
> > > >
> > > > Vote will be open for 72 hours. Please VOTE as follows:
> > > >
> > > > [ ] +1, let's get it released!!!
> > > > [ ] +/-0, fine, but consider to fix few issues before...
> > > > [ ] -1, nope, because... (and please explain why)
> > > >
> > > > Along with your VOTE, please indicate testing and checks you've made
> > > > against build artifacts, src, and documentation:
> > > >
> > > > [ ] Build and Unit Tests Pass
> > > > [ ] Integration Tests Pass
> > > > [ ] "Incubating" in References to Project and Distribution File Names
> > > > [ ] Signatures and Hashes Match Keys
> > > > [ ] DISCLAIMER, LICENSE, and NOTICE Files in Source and Binary
> Release
> > > > Packages
> > > > [ ] DISCLAIMER, LICENSE, and NOTICE are consistent with ASF and
> > Incubator
> > > > Policy
> > > > [ ] CHANGELOG included with release distribution
> > > > [ ] All Source Files Have Correct ASF Headers
> > > > [ ] No Binary Files in 

Re: [VOTE] Release Apache Flagon UserALE.js (Incubating) 2.2.0 (RC3)

2021-06-18 Thread Furkan KAMACI
Hi,

+1 from me (binding, carrying over my vote).

I checked:

- Incubating in name
- DISCLAIMER exists
- LICENSE and NOTICE are fine
- No unexpected binary files
- Checked PGP signatures
- Checked checksums
- Code compiles and tests successfully run

Kind Regards,
Furkan KAMACI

On Fri, Jun 18, 2021 at 8:59 AM Gedd Johnson  wrote:

> Hello Liu!
>
> Thanks for taking a look into this. The files mentioned are in JSON format
> which does not allow for comments. As far as we know, the ASF header is not
> able to be included in this file type. See the linked example from the
> package.json file in Apache Airflow
> .
>
> Best,
> Gedd Johnson
>
> On Fri, Jun 18, 2021 at 3:52 AM Xun Liu  wrote:
>
> > Hi, Johnson
> >
> > I have checked the following items:
> >
> > - Incubating in name
> > - NOTICE is fine
> > - DISCLAIMER exists
> > - All links are valid
> > - No unexpected JS files
> >
> > BUT, This file does not have ASF headers, Please fixed it.
> >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package-lock.json
> >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/cypress.json
> >
> >
> https://github.com/apache/incubator-flagon-useralejs/blob/master/package.json
> >
> > Xun Liu
> > Best regards
> >
> > On Thu, Jun 17, 2021 at 2:42 PM Gedd Johnson 
> > wrote:
> >
> > > Hi Folks,
> > >
> > > Please VOTE on the Apache Flagon UserALE.js 2.2.0 Release Candidate #3.
> > >
> > > A VOTE was carried out on dev@flagon with the following results:
> > >
> > > [ 4 ] +1, let's get it released!!! (+2 (IPMC/Binding) +2 PPMC)
> > > [ 0 ] +/-0, fine, but consider to fix few issues before...
> > > [ 0 ] -1, nope, because... (and please explain why)
> > >
> > > We require +1 IPMC/Binding VOTE to Release.
> > >
> > > The VOTE thread can be found here:
> > >
> > >
> >
> https://lists.apache.org/thread.html/rfa74ac6182a195facddecb4218f12b1354c02a0eee4bbb24d8d4d024%40%3Cdev.flagon.apache.org%3E
> > >  (note
> > > that the thread got a little convoluted, this link reflects the thread
> > for
> > > 2.2.0 RC3)
> > >
> > > About Flagon: http://flagon.incubator.apache.org/
> > >
> > > This Minor release includes:
> > >
> > > 1. Adding a page load log feature
> > > 2. Refactoring the build pipeline using Rollup
> > > 3. Adding a journey testing suite
> > > 4. Fixing a bug with sendOnClose
> > >
> > > We solved 21 issues:
> > > https://github.com/apache/incubator-flagon-useralejs/projects/7
> > >
> > > Git source tag (2b996df):
> > >
> >
> https://github.com/apache/incubator-flagon-useralejs/releases/tag/2.2.0-RC3
> > >
> > > Staging repo: https://dist.apache.org/repos/dist/dev/incubator/flagon/
> > >
> > > Source Release Artifacts:
> > >
> > >
> >
> https://dist.apache.org/repos/dist/dev/incubator/flagon/apache-flagon-useralejs-incubating-2.2.0-RC3/
> > >
> > > PGP release keys (signed using A88C905BD4516D4C):
> > > https://dist.apache.org/repos/dist/dev/incubator/flagon/KEYS
> > >
> > > Link to successful Github CI Build:
> > > https://github.com/apache/incubator-flagon-useralejs/actions
> > >
> > > Reference the useralejs testing framework to assist in your unit and
> > > integration tests:
> > >
> > >
> >
> https://cwiki.apache.org/confluence/display/FLAGON/UserALE.js+Testing+Framework
> > >
> > > Vote will be open for 72 hours. Please VOTE as follows:
> > >
> > > [ ] +1, let's get it released!!!
> > > [ ] +/-0, fine, but consider to fix few issues before...
> > > [ ] -1, nope, because... (and please explain why)
> > >
> > > Along with your VOTE, please indicate testing and checks you've made
> > > against build artifacts, src, and documentation:
> > >
> > > [ ] Build and Unit Tests Pass
> > > [ ] Integration Tests Pass
> > > [ ] "Incubating" in References to Project and Distribution File Names
> > > [ ] Signatures and Hashes Match Keys
> > > [ ] DISCLAIMER, LICENSE, and NOTICE Files in Source and Binary Release
> > > Packages
> > > [ ] DISCLAIMER, LICENSE, and NOTICE are consistent with ASF and
> Incubator
> > > Policy
> > > [ ] CHANGELOG included with release distribution
> > > [ ] All Source Files Have Correct ASF Headers
> > > [ ] No Binary Files in Source Release Packages
> > >
> > > Thank you to everyone that is able to VOTE as well as everyone that
> > > contributed to Apache Flagon UserALE.js (Incubating) 2.2.0
> > >
> >
>