Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Furkan KAMACI]

Hi,

+1 (binding)

Kind Regards,
Furkan KAMACI

On Wed, Apr 14, 2021 at 9:28 AM Fu Xiang  wrote:

> Thanks, Justin,
>
> Which OS and maven version are you using? I just ran all the tests on my
> ubuntu machine, and it's pretty stable. We will go through your
> configuration during next release :P
>
>  Best,
>
> Xiang
>
> On Tue, Apr 13, 2021 at 3:30 PM Justin Mclean 
> wrote:
>
> > Hi,
> >
> > Changing my vote to +1 (binding).
> >
> > > Pinot has a nodejs based UI component under pinot-controller
> > > <
> >
> https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0/pinot-controller/src/main/resources
> > >.
> > > Those dependencies are from the generated directories node_modules/dist
> > > after the npm build, which were previously packaged into src dist and
> > > removed since this version.
> >
> > Thanks for the explanation.
> >
> > > Can I ask for an exception for this release, and I will fix it before
> the
> > > next release?
> >
> > Well I guess having extra information in your LICENSE is not as bad as
> > missing license information. Please correct this for the next release.
> >
> > > Regarding the test failing issue, are you running it using JDK 8?
> >
> > I was using JDK 8 yes.
> >
> > Thanks,
> > Justin
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>
> --
> Xiang Fu
>

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Fu Xiang]

Thanks, Justin,

Which OS and maven version are you using? I just ran all the tests on my
ubuntu machine, and it's pretty stable. We will go through your
configuration during next release :P

 Best,

Xiang

On Tue, Apr 13, 2021 at 3:30 PM Justin Mclean 
wrote:

> Hi,
>
> Changing my vote to +1 (binding).
>
> > Pinot has a nodejs based UI component under pinot-controller
> > <
> https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0/pinot-controller/src/main/resources
> >.
> > Those dependencies are from the generated directories node_modules/dist
> > after the npm build, which were previously packaged into src dist and
> > removed since this version.
>
> Thanks for the explanation.
>
> > Can I ask for an exception for this release, and I will fix it before the
> > next release?
>
> Well I guess having extra information in your LICENSE is not as bad as
> missing license information. Please correct this for the next release.
>
> > Regarding the test failing issue, are you running it using JDK 8?
>
> I was using JDK 8 yes.
>
> Thanks,
> Justin
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

-- 
Xiang Fu

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[kishore g]

+1 (binding)

On Tue, Apr 13, 2021 at 6:41 PM Felix Cheung  wrote:

> Carry over my vote
>
> +1 binding
>
>
> On Sat, Apr 10, 2021 at 4:04 PM Fu Xiang  wrote:
>
> > Hi all,
> >
> > This is a call for a vote to release Apache Pinot (incubating) version
> > 0.7.1.
> >
> > Apache Pinot (incubating) is a distributed columnar storage engine that
> can
> > ingest data in realtime and serve analytical queries at low latency.
> >
> > Pinot community has voted and approved this release.
> >
> > Vote threads:
> >
> >
> https://lists.apache.org/thread.html/rf3737f7739a085fd17120807ea41dfe501137ff26ccc6fa40f96ae4a%40%3Cdev.pinot.apache.org%3E
> >
> > Result thread:
> >
> >
> https://lists.apache.org/thread.html/ra630b726e8a8e2a449555d75cfbe876a7b04a9f10880ea1ea5bb%40%3Cdev.pinot.apache.org%3E
> >
> > The release candidate:
> > https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-
> > incubating-0.7.1-rc0/
> > <
> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-incubating-0.7.1-rc0/
> >
> >
> > Git tag for this release:
> > https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0
> >
> > Git hash for this release:
> > *e22be7c3a39e840321d3658e7505f21768b228d6*
> >
> > The artifacts have been signed with key: *CDEDB21B862F6C66*, which can be
> > found in the following KEYS file.
> > https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS
> >
> > Release notes:
> > https://github.com/apache/incubator-pinot/releases/tag/release-0.7.1-rc0
> >
> > Staging repository:
> > https://repository.apache.org/content/repositories/orgapachepinot-1025
> >
> > Documentation on verifying a release candidate:
> > https://cwiki.apache.org/confluence/display/PINOT
> > /Validating+a+release+candidate
> >
> > The vote will be open for at least 72 hours or until the necessary number
> > of votes are reached.
> >
> > Please vote accordingly,
> >
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove with the reason
> >
> > Thanks,
> >
> > Apache Pinot (incubating) team
> >
>

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Felix Cheung]

Carry over my vote

+1 binding


On Sat, Apr 10, 2021 at 4:04 PM Fu Xiang  wrote:

> Hi all,
>
> This is a call for a vote to release Apache Pinot (incubating) version
> 0.7.1.
>
> Apache Pinot (incubating) is a distributed columnar storage engine that can
> ingest data in realtime and serve analytical queries at low latency.
>
> Pinot community has voted and approved this release.
>
> Vote threads:
>
> https://lists.apache.org/thread.html/rf3737f7739a085fd17120807ea41dfe501137ff26ccc6fa40f96ae4a%40%3Cdev.pinot.apache.org%3E
>
> Result thread:
>
> https://lists.apache.org/thread.html/ra630b726e8a8e2a449555d75cfbe876a7b04a9f10880ea1ea5bb%40%3Cdev.pinot.apache.org%3E
>
> The release candidate:
> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-
> incubating-0.7.1-rc0/
> 
>
> Git tag for this release:
> https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0
>
> Git hash for this release:
> *e22be7c3a39e840321d3658e7505f21768b228d6*
>
> The artifacts have been signed with key: *CDEDB21B862F6C66*, which can be
> found in the following KEYS file.
> https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS
>
> Release notes:
> https://github.com/apache/incubator-pinot/releases/tag/release-0.7.1-rc0
>
> Staging repository:
> https://repository.apache.org/content/repositories/orgapachepinot-1025
>
> Documentation on verifying a release candidate:
> https://cwiki.apache.org/confluence/display/PINOT
> /Validating+a+release+candidate
>
> The vote will be open for at least 72 hours or until the necessary number
> of votes are reached.
>
> Please vote accordingly,
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thanks,
>
> Apache Pinot (incubating) team
>

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Justin Mclean]

Hi,

Changing my vote to +1 (binding).

> Pinot has a nodejs based UI component under pinot-controller
> .
> Those dependencies are from the generated directories node_modules/dist
> after the npm build, which were previously packaged into src dist and
> removed since this version.

Thanks for the explanation.

> Can I ask for an exception for this release, and I will fix it before the
> next release?

Well I guess having extra information in your LICENSE is not as bad as missing 
license information. Please correct this for the next release.

> Regarding the test failing issue, are you running it using JDK 8?

I was using JDK 8 yes.

Thanks,
Justin
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Seunghyun Lee]

Also, we have created a Github issue to address this asap.
https://github.com/apache/incubator-pinot/issues/6785

On Tue, Apr 13, 2021 at 12:15 PM Fu Xiang  wrote:
>
> Thanks, Justin,
>
> Pinot has a nodejs based UI component under pinot-controller
> .
> Those dependencies are from the generated directories node_modules/dist
> after the npm build, which were previously packaged into src dist and
> removed since this version.
>
> Can I ask for an exception for this release, and I will fix it before the
> next release?
>
> Regarding the test failing issue, are you running it using JDK 8?
>
>
>
> On Mon, Apr 12, 2021 at 11:45 PM Justin Mclean 
> wrote:
>
> > Hi,
> >
> > -1 (binding) due to LICENSE file
> >
> > I very confused by the license file as it seem to be included dependancies
> > and tools used and not just what is bundled in the source release? [1] How
> > was this list of software and tools generated?
> >
> > I checked:
> > - incubating in name
> > - disclaimer exits
> > - LICENSE seem to include software that is not bundled in the source
> > release
> > - NOTICE is fine
> > - DISCLAIMER exists
> > - no unexpected binary files
> > - all ASF files have ASF headers
> >
> > I not able to compile from source but it probably by setup, either I get
> > test failing or the "Pinot Integration Tests” stage failing.
> >
> > Thanks,
> > Justin
> >
> > 1.https://infra.apache.org/licensing-howto.html#guiding
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>
> --
> Xiang Fu

-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Fu Xiang]

Thanks, Justin,

Pinot has a nodejs based UI component under pinot-controller
.
Those dependencies are from the generated directories node_modules/dist
after the npm build, which were previously packaged into src dist and
removed since this version.

Can I ask for an exception for this release, and I will fix it before the
next release?

Regarding the test failing issue, are you running it using JDK 8?



On Mon, Apr 12, 2021 at 11:45 PM Justin Mclean 
wrote:

> Hi,
>
> -1 (binding) due to LICENSE file
>
> I very confused by the license file as it seem to be included dependancies
> and tools used and not just what is bundled in the source release? [1] How
> was this list of software and tools generated?
>
> I checked:
> - incubating in name
> - disclaimer exits
> - LICENSE seem to include software that is not bundled in the source
> release
> - NOTICE is fine
> - DISCLAIMER exists
> - no unexpected binary files
> - all ASF files have ASF headers
>
> I not able to compile from source but it probably by setup, either I get
> test failing or the "Pinot Integration Tests” stage failing.
>
> Thanks,
> Justin
>
> 1.https://infra.apache.org/licensing-howto.html#guiding
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

-- 
Xiang Fu

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Justin Mclean]

Hi,

-1 (binding) due to LICENSE file

I very confused by the license file as it seem to be included dependancies and 
tools used and not just what is bundled in the source release? [1] How was this 
list of software and tools generated?

I checked:
- incubating in name
- disclaimer exits
- LICENSE seem to include software that is not bundled in the source release
- NOTICE is fine
- DISCLAIMER exists
- no unexpected binary files
- all ASF files have ASF headers

I not able to compile from source but it probably by setup, either I get test 
failing or the "Pinot Integration Tests” stage failing. 

Thanks,
Justin

1.https://infra.apache.org/licensing-howto.html#guiding
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Xun Liu]

> Added this into the Validation document.

OK,
+1 (non-binding)

Thanks,
Xun Liu

On Mon, Apr 12, 2021 at 3:24 PM Fu Xiang  wrote:

> Hi Xun,
>
> Yes, Pinot requires JDK 8 to compile and build.
>
> Added this into the Validation document.
>
> Best,
>
> Xiang
>
> On Sat, Apr 10, 2021 at 7:03 PM Xun Liu  wrote:
>
> > I found 0.7.1 RC0 that it can only compile correctly when JDK > 8 or
> above.
> > Do we explain this in the documentation? Maybe I missed it.
> >
> > I check
> > [√] “incubating" included in the name
> > [√] Checked LICENSE, NOTICE, and DISCLAIMER
> > [√] Source files have ASF header.
> >
> > On Sun, Apr 11, 2021 at 7:04 AM Fu Xiang  wrote:
> >
> > > Hi all,
> > >
> > > This is a call for a vote to release Apache Pinot (incubating) version
> > > 0.7.1.
> > >
> > > Apache Pinot (incubating) is a distributed columnar storage engine that
> > can
> > > ingest data in realtime and serve analytical queries at low latency.
> > >
> > > Pinot community has voted and approved this release.
> > >
> > > Vote threads:
> > >
> > >
> >
> https://lists.apache.org/thread.html/rf3737f7739a085fd17120807ea41dfe501137ff26ccc6fa40f96ae4a%40%3Cdev.pinot.apache.org%3E
> > >
> > > Result thread:
> > >
> > >
> >
> https://lists.apache.org/thread.html/ra630b726e8a8e2a449555d75cfbe876a7b04a9f10880ea1ea5bb%40%3Cdev.pinot.apache.org%3E
> > >
> > > The release candidate:
> > > https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-
> > > incubating-0.7.1-rc0/
> > > <
> >
> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-incubating-0.7.1-rc0/
> > >
> > >
> > > Git tag for this release:
> > > https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0
> > >
> > > Git hash for this release:
> > > *e22be7c3a39e840321d3658e7505f21768b228d6*
> > >
> > > The artifacts have been signed with key: *CDEDB21B862F6C66*, which can
> be
> > > found in the following KEYS file.
> > > https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS
> > >
> > > Release notes:
> > >
> https://github.com/apache/incubator-pinot/releases/tag/release-0.7.1-rc0
> > >
> > > Staging repository:
> > > https://repository.apache.org/content/repositories/orgapachepinot-1025
> > >
> > > Documentation on verifying a release candidate:
> > > https://cwiki.apache.org/confluence/display/PINOT
> > > /Validating+a+release+candidate
> > >
> > > The vote will be open for at least 72 hours or until the necessary
> number
> > > of votes are reached.
> > >
> > > Please vote accordingly,
> > >
> > > [ ] +1 approve
> > > [ ] +0 no opinion
> > > [ ] -1 disapprove with the reason
> > >
> > > Thanks,
> > >
> > > Apache Pinot (incubating) team
> > >
> >
>
>
> --
> Xiang Fu
>

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Fu Xiang]

Hi Xun,

Yes, Pinot requires JDK 8 to compile and build.

Added this into the Validation document.

Best,

Xiang

On Sat, Apr 10, 2021 at 7:03 PM Xun Liu  wrote:

> I found 0.7.1 RC0 that it can only compile correctly when JDK > 8 or above.
> Do we explain this in the documentation? Maybe I missed it.
>
> I check
> [√] “incubating" included in the name
> [√] Checked LICENSE, NOTICE, and DISCLAIMER
> [√] Source files have ASF header.
>
> On Sun, Apr 11, 2021 at 7:04 AM Fu Xiang  wrote:
>
> > Hi all,
> >
> > This is a call for a vote to release Apache Pinot (incubating) version
> > 0.7.1.
> >
> > Apache Pinot (incubating) is a distributed columnar storage engine that
> can
> > ingest data in realtime and serve analytical queries at low latency.
> >
> > Pinot community has voted and approved this release.
> >
> > Vote threads:
> >
> >
> https://lists.apache.org/thread.html/rf3737f7739a085fd17120807ea41dfe501137ff26ccc6fa40f96ae4a%40%3Cdev.pinot.apache.org%3E
> >
> > Result thread:
> >
> >
> https://lists.apache.org/thread.html/ra630b726e8a8e2a449555d75cfbe876a7b04a9f10880ea1ea5bb%40%3Cdev.pinot.apache.org%3E
> >
> > The release candidate:
> > https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-
> > incubating-0.7.1-rc0/
> > <
> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-incubating-0.7.1-rc0/
> >
> >
> > Git tag for this release:
> > https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0
> >
> > Git hash for this release:
> > *e22be7c3a39e840321d3658e7505f21768b228d6*
> >
> > The artifacts have been signed with key: *CDEDB21B862F6C66*, which can be
> > found in the following KEYS file.
> > https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS
> >
> > Release notes:
> > https://github.com/apache/incubator-pinot/releases/tag/release-0.7.1-rc0
> >
> > Staging repository:
> > https://repository.apache.org/content/repositories/orgapachepinot-1025
> >
> > Documentation on verifying a release candidate:
> > https://cwiki.apache.org/confluence/display/PINOT
> > /Validating+a+release+candidate
> >
> > The vote will be open for at least 72 hours or until the necessary number
> > of votes are reached.
> >
> > Please vote accordingly,
> >
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove with the reason
> >
> > Thanks,
> >
> > Apache Pinot (incubating) team
> >
>


-- 
Xiang Fu

Re: [VOTE] Release Apache Pinot (incubating) 0.7.1 RC0

[Xun Liu]

I found 0.7.1 RC0 that it can only compile correctly when JDK > 8 or above.
Do we explain this in the documentation? Maybe I missed it.

I check
[√] “incubating" included in the name
[√] Checked LICENSE, NOTICE, and DISCLAIMER
[√] Source files have ASF header.

On Sun, Apr 11, 2021 at 7:04 AM Fu Xiang  wrote:

> Hi all,
>
> This is a call for a vote to release Apache Pinot (incubating) version
> 0.7.1.
>
> Apache Pinot (incubating) is a distributed columnar storage engine that can
> ingest data in realtime and serve analytical queries at low latency.
>
> Pinot community has voted and approved this release.
>
> Vote threads:
>
> https://lists.apache.org/thread.html/rf3737f7739a085fd17120807ea41dfe501137ff26ccc6fa40f96ae4a%40%3Cdev.pinot.apache.org%3E
>
> Result thread:
>
> https://lists.apache.org/thread.html/ra630b726e8a8e2a449555d75cfbe876a7b04a9f10880ea1ea5bb%40%3Cdev.pinot.apache.org%3E
>
> The release candidate:
> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-
> incubating-0.7.1-rc0/
> 
>
> Git tag for this release:
> https://github.com/apache/incubator-pinot/tree/release-0.7.1-rc0
>
> Git hash for this release:
> *e22be7c3a39e840321d3658e7505f21768b228d6*
>
> The artifacts have been signed with key: *CDEDB21B862F6C66*, which can be
> found in the following KEYS file.
> https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS
>
> Release notes:
> https://github.com/apache/incubator-pinot/releases/tag/release-0.7.1-rc0
>
> Staging repository:
> https://repository.apache.org/content/repositories/orgapachepinot-1025
>
> Documentation on verifying a release candidate:
> https://cwiki.apache.org/confluence/display/PINOT
> /Validating+a+release+candidate
>
> The vote will be open for at least 72 hours or until the necessary number
> of votes are reached.
>
> Please vote accordingly,
>
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thanks,
>
> Apache Pinot (incubating) team
>