Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-16 Thread Dongjoon Hyun
Thank you all.

ORC-1476 was merged to Apache ORC 2.0 as an improvement for robust building.
ORC-1475 was merged to Apache ORC 2.0 and 1.9.2 as a `Test`-type JIRA commit.

This vote passed. I'm going to conclude this vote.

Dongjoon.

On 2023/08/15 17:38:37 Dongjoon Hyun wrote:
> Thank you, Gang and Yiqun.
> 
> 1. For https://issues.apache.org/jira/browse/ORC-1475, I switched the type 
> from `Bug` to `Test` because it's only a change on TestConvertColumnReader.cc.
> 
> 2. For https://issues.apache.org/jira/browse/ORC-1476, I'm also M1 Max on Mac 
> user and this RC0 was published from my M1 Max notebook with Java 8 with all 
> Unit Test success. I added my comments on the PRs about ORC-1476 to 
> understand the situation.
> 
> Dongjoon
> 
> On 2023/08/13 16:07:32 Yiqun Zhang wrote:
> > +1
> > 
> > - Built and tested C++/Java on macOS (intel) 13.4.1 and openjdk 17.0.8
> > - Verified checksum and signature
> > 
> > Thanks,
> > Yiqun
> > 
> > Gang Wu  于2023年8月13日周日 09:46写道:
> > 
> > > Filed two jiras:
> > >
> > > https://issues.apache.org/jira/browse/ORC-1475
> > > https://issues.apache.org/jira/browse/ORC-1476
> > >
> > > On Sun, Aug 13, 2023 at 4:33 AM Dongjoon Hyun 
> > > wrote:
> > >
> > > > Thank you, all.
> > > >
> > > > For the reported cases, please file the corresponding Jira issues if 
> > > > they
> > > > don’t have already.
> > > >
> > > > Dongjoon
> > > >
> > > > On Sat, Aug 12, 2023 at 10:39 William H.  wrote:
> > > >
> > > > > +1
> > > > >
> > > > > Thank you Dongjoon for driving this release.
> > > > >
> > > > > Bests,
> > > > > William
> > > > >
> > > > > On Sat, Aug 12, 2023 at 9:18 AM mystic lama 
> > > > > wrote:
> > > > >
> > > > > > @Gang Wu - Thanks for the explanation. Makes sense to me.
> > > > > >
> > > > > > My vote +1
> > > > > > - verified checksum and signature
> > > > > > - ran Java build, compilation and tests successful on M1 mac
> > > > > > - All JIRA's look good
> > > > > >
> > > > > > Minor observation(s)
> > > > > > - All PR's are in closed state, but the following 2 are in merged
> > > > state.
> > > > > > Just an observation
> > > > > >   - https://github.com/apache/orc/pull/1551
> > > > > >   - https://github.com/apache/orc/pull/1552
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
> > > > > >
> > > > > > > +1
> > > > > > >
> > > > > > > To Mystic:
> > > > > > > The root cause is that the compiler uses signed char to interpret
> > > > > > > char. The error may disappear if -funsigned-char is added to the
> > > > > > > compiler options. I will take a look to eliminate the UB issue but
> > > > > > > I do not think this is a blocker to the release.
> > > > > > >
> > > > > > >
> > > > > > > Verified checksum and signature.
> > > > > > > All Java and C++ build and test passed on my Mac locally.
> > > > > > > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> > > > > > >
> > > > > > > BTW, I initially failed with the following error:
> > > > > > > [ERROR] Failed to execute goal
> > > > > > > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on
> > > > project
> > > > > > > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> > > > > > platform:
> > > > > > > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > > > > > > I did the trick from link [1] and it passed.
> > > > > > >
> > > > > > > [1]
> > > > > https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> > > > > > >
> > > > > > > Best,
> > > > > > > Gang
> > > > > > >
> > > > > > >
> > > > > > > On Sat, Aug 12, 2023 at 12:56 PM mystic lama <
> > > > mysticlama...@gmail.com>
> > > > > > > wrote:
> > > > > > >
> > > > > > > > I am getting an error when I ran docker test on my local machine
> > > > like
> > > > > > > > It could be a local issue, but just want to confirm before
> > > sending
> > > > in
> > > > > > my
> > > > > > > > vote
> > > > > > > >
> > > > > > > > Error from log towards end of email.
> > > > > > > >
> > > > > > > > Steps I followed
> > > > > > > > 1. Downloaded from
> > > > > > > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > > > > > > 2. After extracting, ran the command from docker folder
> > > > > > > >
> > > > > > > > bash-5.1$ ./run-all.sh local branch-1.9
> > > > > > > > Launching centos7 as 85438
> > > > > > > > Launching debian10 as 85439
> > > > > > > > Launching debian11 as 85440
> > > > > > > > Launching ubuntu20 as 85441
> > > > > > > > Launching ubuntu22 as 85443
> > > > > > > > Launching fedora37 as 85448
> > > > > > > > Launching debian11_jdk=11 as 85454
> > > > > > > > Launching ubuntu22_jdk=11 as 85461
> > > > > > > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > > > > > > Waiting for 85438
> > > > > > > > Failed tests
> > > > > > > > [  FAILED  ]
> > > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > (64
> > > > > > > ms)
> > > > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > > > [  FAILED  ]
> > > ConvertColumnReader.TestConvertNumericToStringVariant

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-15 Thread Dongjoon Hyun
Thank you, Gang and Yiqun.

1. For https://issues.apache.org/jira/browse/ORC-1475, I switched the type from 
`Bug` to `Test` because it's only a change on TestConvertColumnReader.cc.

2. For https://issues.apache.org/jira/browse/ORC-1476, I'm also M1 Max on Mac 
user and this RC0 was published from my M1 Max notebook with Java 8 with all 
Unit Test success. I added my comments on the PRs about ORC-1476 to understand 
the situation.

Dongjoon

On 2023/08/13 16:07:32 Yiqun Zhang wrote:
> +1
> 
> - Built and tested C++/Java on macOS (intel) 13.4.1 and openjdk 17.0.8
> - Verified checksum and signature
> 
> Thanks,
> Yiqun
> 
> Gang Wu  于2023年8月13日周日 09:46写道:
> 
> > Filed two jiras:
> >
> > https://issues.apache.org/jira/browse/ORC-1475
> > https://issues.apache.org/jira/browse/ORC-1476
> >
> > On Sun, Aug 13, 2023 at 4:33 AM Dongjoon Hyun 
> > wrote:
> >
> > > Thank you, all.
> > >
> > > For the reported cases, please file the corresponding Jira issues if they
> > > don’t have already.
> > >
> > > Dongjoon
> > >
> > > On Sat, Aug 12, 2023 at 10:39 William H.  wrote:
> > >
> > > > +1
> > > >
> > > > Thank you Dongjoon for driving this release.
> > > >
> > > > Bests,
> > > > William
> > > >
> > > > On Sat, Aug 12, 2023 at 9:18 AM mystic lama 
> > > > wrote:
> > > >
> > > > > @Gang Wu - Thanks for the explanation. Makes sense to me.
> > > > >
> > > > > My vote +1
> > > > > - verified checksum and signature
> > > > > - ran Java build, compilation and tests successful on M1 mac
> > > > > - All JIRA's look good
> > > > >
> > > > > Minor observation(s)
> > > > > - All PR's are in closed state, but the following 2 are in merged
> > > state.
> > > > > Just an observation
> > > > >   - https://github.com/apache/orc/pull/1551
> > > > >   - https://github.com/apache/orc/pull/1552
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
> > > > >
> > > > > > +1
> > > > > >
> > > > > > To Mystic:
> > > > > > The root cause is that the compiler uses signed char to interpret
> > > > > > char. The error may disappear if -funsigned-char is added to the
> > > > > > compiler options. I will take a look to eliminate the UB issue but
> > > > > > I do not think this is a blocker to the release.
> > > > > >
> > > > > >
> > > > > > Verified checksum and signature.
> > > > > > All Java and C++ build and test passed on my Mac locally.
> > > > > > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> > > > > >
> > > > > > BTW, I initially failed with the following error:
> > > > > > [ERROR] Failed to execute goal
> > > > > > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on
> > > project
> > > > > > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> > > > > platform:
> > > > > > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > > > > > I did the trick from link [1] and it passed.
> > > > > >
> > > > > > [1]
> > > > https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> > > > > >
> > > > > > Best,
> > > > > > Gang
> > > > > >
> > > > > >
> > > > > > On Sat, Aug 12, 2023 at 12:56 PM mystic lama <
> > > mysticlama...@gmail.com>
> > > > > > wrote:
> > > > > >
> > > > > > > I am getting an error when I ran docker test on my local machine
> > > like
> > > > > > > It could be a local issue, but just want to confirm before
> > sending
> > > in
> > > > > my
> > > > > > > vote
> > > > > > >
> > > > > > > Error from log towards end of email.
> > > > > > >
> > > > > > > Steps I followed
> > > > > > > 1. Downloaded from
> > > > > > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > > > > > 2. After extracting, ran the command from docker folder
> > > > > > >
> > > > > > > bash-5.1$ ./run-all.sh local branch-1.9
> > > > > > > Launching centos7 as 85438
> > > > > > > Launching debian10 as 85439
> > > > > > > Launching debian11 as 85440
> > > > > > > Launching ubuntu20 as 85441
> > > > > > > Launching ubuntu22 as 85443
> > > > > > > Launching fedora37 as 85448
> > > > > > > Launching debian11_jdk=11 as 85454
> > > > > > > Launching ubuntu22_jdk=11 as 85461
> > > > > > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > > > > > Waiting for 85438
> > > > > > > Failed tests
> > > > > > > [  FAILED  ]
> > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > (64
> > > > > > ms)
> > > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > > [  FAILED  ]
> > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > > >  1 FAILED TEST
> > > > > > > FAILED centos7
> > > > > > > [  FAILED  ]
> > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > (19
> > > > > > ms)
> > > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > > [  FAILED  ]
> > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > > >  1 FAILED TEST
> > > > > > > FAILED debian10
> > > > > > > [  FAILED  ]
> > ConvertColumnReader.TestConvertNumericToStringVariant
> > > > (62
> > > > > > ms)
> > > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > > [  FAILED  ]
> > 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-13 Thread Yiqun Zhang
+1

- Built and tested C++/Java on macOS (intel) 13.4.1 and openjdk 17.0.8
- Verified checksum and signature

Thanks,
Yiqun

Gang Wu  于2023年8月13日周日 09:46写道:

> Filed two jiras:
>
> https://issues.apache.org/jira/browse/ORC-1475
> https://issues.apache.org/jira/browse/ORC-1476
>
> On Sun, Aug 13, 2023 at 4:33 AM Dongjoon Hyun 
> wrote:
>
> > Thank you, all.
> >
> > For the reported cases, please file the corresponding Jira issues if they
> > don’t have already.
> >
> > Dongjoon
> >
> > On Sat, Aug 12, 2023 at 10:39 William H.  wrote:
> >
> > > +1
> > >
> > > Thank you Dongjoon for driving this release.
> > >
> > > Bests,
> > > William
> > >
> > > On Sat, Aug 12, 2023 at 9:18 AM mystic lama 
> > > wrote:
> > >
> > > > @Gang Wu - Thanks for the explanation. Makes sense to me.
> > > >
> > > > My vote +1
> > > > - verified checksum and signature
> > > > - ran Java build, compilation and tests successful on M1 mac
> > > > - All JIRA's look good
> > > >
> > > > Minor observation(s)
> > > > - All PR's are in closed state, but the following 2 are in merged
> > state.
> > > > Just an observation
> > > >   - https://github.com/apache/orc/pull/1551
> > > >   - https://github.com/apache/orc/pull/1552
> > > >
> > > >
> > > >
> > > >
> > > > On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
> > > >
> > > > > +1
> > > > >
> > > > > To Mystic:
> > > > > The root cause is that the compiler uses signed char to interpret
> > > > > char. The error may disappear if -funsigned-char is added to the
> > > > > compiler options. I will take a look to eliminate the UB issue but
> > > > > I do not think this is a blocker to the release.
> > > > >
> > > > >
> > > > > Verified checksum and signature.
> > > > > All Java and C++ build and test passed on my Mac locally.
> > > > > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> > > > >
> > > > > BTW, I initially failed with the following error:
> > > > > [ERROR] Failed to execute goal
> > > > > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on
> > project
> > > > > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> > > > platform:
> > > > > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > > > > I did the trick from link [1] and it passed.
> > > > >
> > > > > [1]
> > > https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> > > > >
> > > > > Best,
> > > > > Gang
> > > > >
> > > > >
> > > > > On Sat, Aug 12, 2023 at 12:56 PM mystic lama <
> > mysticlama...@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > I am getting an error when I ran docker test on my local machine
> > like
> > > > > > It could be a local issue, but just want to confirm before
> sending
> > in
> > > > my
> > > > > > vote
> > > > > >
> > > > > > Error from log towards end of email.
> > > > > >
> > > > > > Steps I followed
> > > > > > 1. Downloaded from
> > > > > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > > > > 2. After extracting, ran the command from docker folder
> > > > > >
> > > > > > bash-5.1$ ./run-all.sh local branch-1.9
> > > > > > Launching centos7 as 85438
> > > > > > Launching debian10 as 85439
> > > > > > Launching debian11 as 85440
> > > > > > Launching ubuntu20 as 85441
> > > > > > Launching ubuntu22 as 85443
> > > > > > Launching fedora37 as 85448
> > > > > > Launching debian11_jdk=11 as 85454
> > > > > > Launching ubuntu22_jdk=11 as 85461
> > > > > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > > > > Waiting for 85438
> > > > > > Failed tests
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (64
> > > > > ms)
> > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > >  1 FAILED TEST
> > > > > > FAILED centos7
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (19
> > > > > ms)
> > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > >  1 FAILED TEST
> > > > > > FAILED debian10
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (62
> > > > > ms)
> > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > >  1 FAILED TEST
> > > > > > FAILED debian11
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (42
> > > > > ms)
> > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > >  1 FAILED TEST
> > > > > > FAILED debian11_jdk=11
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (58
> > > > > ms)
> > > > > > [  FAILED  ] 1 test, listed below:
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > > > >  1 FAILED TEST
> > > > > > [  FAILED  ]
> ConvertColumnReader.TestConvertNumericToStringVariant
> > > (26
> > > > > ms)
> > 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-12 Thread Gang Wu
Filed two jiras:

https://issues.apache.org/jira/browse/ORC-1475
https://issues.apache.org/jira/browse/ORC-1476

On Sun, Aug 13, 2023 at 4:33 AM Dongjoon Hyun 
wrote:

> Thank you, all.
>
> For the reported cases, please file the corresponding Jira issues if they
> don’t have already.
>
> Dongjoon
>
> On Sat, Aug 12, 2023 at 10:39 William H.  wrote:
>
> > +1
> >
> > Thank you Dongjoon for driving this release.
> >
> > Bests,
> > William
> >
> > On Sat, Aug 12, 2023 at 9:18 AM mystic lama 
> > wrote:
> >
> > > @Gang Wu - Thanks for the explanation. Makes sense to me.
> > >
> > > My vote +1
> > > - verified checksum and signature
> > > - ran Java build, compilation and tests successful on M1 mac
> > > - All JIRA's look good
> > >
> > > Minor observation(s)
> > > - All PR's are in closed state, but the following 2 are in merged
> state.
> > > Just an observation
> > >   - https://github.com/apache/orc/pull/1551
> > >   - https://github.com/apache/orc/pull/1552
> > >
> > >
> > >
> > >
> > > On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
> > >
> > > > +1
> > > >
> > > > To Mystic:
> > > > The root cause is that the compiler uses signed char to interpret
> > > > char. The error may disappear if -funsigned-char is added to the
> > > > compiler options. I will take a look to eliminate the UB issue but
> > > > I do not think this is a blocker to the release.
> > > >
> > > >
> > > > Verified checksum and signature.
> > > > All Java and C++ build and test passed on my Mac locally.
> > > > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> > > >
> > > > BTW, I initially failed with the following error:
> > > > [ERROR] Failed to execute goal
> > > > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on
> project
> > > > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> > > platform:
> > > > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > > > I did the trick from link [1] and it passed.
> > > >
> > > > [1]
> > https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> > > >
> > > > Best,
> > > > Gang
> > > >
> > > >
> > > > On Sat, Aug 12, 2023 at 12:56 PM mystic lama <
> mysticlama...@gmail.com>
> > > > wrote:
> > > >
> > > > > I am getting an error when I ran docker test on my local machine
> like
> > > > > It could be a local issue, but just want to confirm before sending
> in
> > > my
> > > > > vote
> > > > >
> > > > > Error from log towards end of email.
> > > > >
> > > > > Steps I followed
> > > > > 1. Downloaded from
> > > > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > > > 2. After extracting, ran the command from docker folder
> > > > >
> > > > > bash-5.1$ ./run-all.sh local branch-1.9
> > > > > Launching centos7 as 85438
> > > > > Launching debian10 as 85439
> > > > > Launching debian11 as 85440
> > > > > Launching ubuntu20 as 85441
> > > > > Launching ubuntu22 as 85443
> > > > > Launching fedora37 as 85448
> > > > > Launching debian11_jdk=11 as 85454
> > > > > Launching ubuntu22_jdk=11 as 85461
> > > > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > > > Waiting for 85438
> > > > > Failed tests
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (64
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > FAILED centos7
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (19
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > FAILED debian10
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (62
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > FAILED debian11
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (42
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > FAILED debian11_jdk=11
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (58
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (26
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > (32
> > > > ms)
> > > > > [  FAILED  ] 1 test, listed below:
> > > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > > >  1 FAILED TEST
> > > > > FAILED ubuntu22
> > > > > [  FAILED  ] 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-12 Thread Dongjoon Hyun
Thank you, all.

For the reported cases, please file the corresponding Jira issues if they
don’t have already.

Dongjoon

On Sat, Aug 12, 2023 at 10:39 William H.  wrote:

> +1
>
> Thank you Dongjoon for driving this release.
>
> Bests,
> William
>
> On Sat, Aug 12, 2023 at 9:18 AM mystic lama 
> wrote:
>
> > @Gang Wu - Thanks for the explanation. Makes sense to me.
> >
> > My vote +1
> > - verified checksum and signature
> > - ran Java build, compilation and tests successful on M1 mac
> > - All JIRA's look good
> >
> > Minor observation(s)
> > - All PR's are in closed state, but the following 2 are in merged state.
> > Just an observation
> >   - https://github.com/apache/orc/pull/1551
> >   - https://github.com/apache/orc/pull/1552
> >
> >
> >
> >
> > On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
> >
> > > +1
> > >
> > > To Mystic:
> > > The root cause is that the compiler uses signed char to interpret
> > > char. The error may disappear if -funsigned-char is added to the
> > > compiler options. I will take a look to eliminate the UB issue but
> > > I do not think this is a blocker to the release.
> > >
> > >
> > > Verified checksum and signature.
> > > All Java and C++ build and test passed on my Mac locally.
> > > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> > >
> > > BTW, I initially failed with the following error:
> > > [ERROR] Failed to execute goal
> > > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project
> > > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> > platform:
> > > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > > I did the trick from link [1] and it passed.
> > >
> > > [1]
> https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> > >
> > > Best,
> > > Gang
> > >
> > >
> > > On Sat, Aug 12, 2023 at 12:56 PM mystic lama 
> > > wrote:
> > >
> > > > I am getting an error when I ran docker test on my local machine like
> > > > It could be a local issue, but just want to confirm before sending in
> > my
> > > > vote
> > > >
> > > > Error from log towards end of email.
> > > >
> > > > Steps I followed
> > > > 1. Downloaded from
> > > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > > 2. After extracting, ran the command from docker folder
> > > >
> > > > bash-5.1$ ./run-all.sh local branch-1.9
> > > > Launching centos7 as 85438
> > > > Launching debian10 as 85439
> > > > Launching debian11 as 85440
> > > > Launching ubuntu20 as 85441
> > > > Launching ubuntu22 as 85443
> > > > Launching fedora37 as 85448
> > > > Launching debian11_jdk=11 as 85454
> > > > Launching ubuntu22_jdk=11 as 85461
> > > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > > Waiting for 85438
> > > > Failed tests
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (64
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED centos7
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (19
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED debian10
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (62
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED debian11
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (42
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED debian11_jdk=11
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (58
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (26
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (32
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED ubuntu22
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (35
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> (53
> > > ms)
> > > > [  FAILED  ] 1 test, listed below:
> > > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > > >  1 FAILED TEST
> > > > FAILED ubuntu22_jdk=11_cc=clang
> > > >
> > > > 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-12 Thread William H.
+1

Thank you Dongjoon for driving this release.

Bests,
William

On Sat, Aug 12, 2023 at 9:18 AM mystic lama  wrote:

> @Gang Wu - Thanks for the explanation. Makes sense to me.
>
> My vote +1
> - verified checksum and signature
> - ran Java build, compilation and tests successful on M1 mac
> - All JIRA's look good
>
> Minor observation(s)
> - All PR's are in closed state, but the following 2 are in merged state.
> Just an observation
>   - https://github.com/apache/orc/pull/1551
>   - https://github.com/apache/orc/pull/1552
>
>
>
>
> On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:
>
> > +1
> >
> > To Mystic:
> > The root cause is that the compiler uses signed char to interpret
> > char. The error may disappear if -funsigned-char is added to the
> > compiler options. I will take a look to eliminate the UB issue but
> > I do not think this is a blocker to the release.
> >
> >
> > Verified checksum and signature.
> > All Java and C++ build and test passed on my Mac locally.
> > Tool versions: openjdk 1.8.0_322 and clang 14.0.6
> >
> > BTW, I initially failed with the following error:
> > [ERROR] Failed to execute goal
> > com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project
> > orc-core: Error extracting protoc for version 3.17.3: Unsupported
> platform:
> > protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> > I did the trick from link [1] and it passed.
> >
> > [1] https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
> >
> > Best,
> > Gang
> >
> >
> > On Sat, Aug 12, 2023 at 12:56 PM mystic lama 
> > wrote:
> >
> > > I am getting an error when I ran docker test on my local machine like
> > > It could be a local issue, but just want to confirm before sending in
> my
> > > vote
> > >
> > > Error from log towards end of email.
> > >
> > > Steps I followed
> > > 1. Downloaded from
> > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > > 2. After extracting, ran the command from docker folder
> > >
> > > bash-5.1$ ./run-all.sh local branch-1.9
> > > Launching centos7 as 85438
> > > Launching debian10 as 85439
> > > Launching debian11 as 85440
> > > Launching ubuntu20 as 85441
> > > Launching ubuntu22 as 85443
> > > Launching fedora37 as 85448
> > > Launching debian11_jdk=11 as 85454
> > > Launching ubuntu22_jdk=11 as 85461
> > > Launching ubuntu22_jdk=11_cc=clang as 85468
> > > Waiting for 85438
> > > Failed tests
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (64
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED centos7
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (19
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED debian10
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (62
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED debian11
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (42
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED debian11_jdk=11
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (58
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (26
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (32
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED ubuntu22
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (35
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (53
> > ms)
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > >  1 FAILED TEST
> > > FAILED ubuntu22_jdk=11_cc=clang
> > >
> > > =
> > > Error from log
> > > =
> > >
> > > [--] 6 tests from ConvertColumnReader
> > > [ RUN  ] ConvertColumnReader.betweenNumericWithoutOverflows
> > > [   OK ] ConvertColumnReader.betweenNumericWithoutOverflows (7 ms)
> > > [ RUN  ] ConvertColumnReader.betweenNumricOverflows
> > > [   OK ] ConvertColumnReader.betweenNumricOverflows (0 ms)
> > > [ RUN  ] 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-12 Thread mystic lama
@Gang Wu - Thanks for the explanation. Makes sense to me.

My vote +1
- verified checksum and signature
- ran Java build, compilation and tests successful on M1 mac
- All JIRA's look good

Minor observation(s)
- All PR's are in closed state, but the following 2 are in merged state.
Just an observation
  - https://github.com/apache/orc/pull/1551
  - https://github.com/apache/orc/pull/1552




On Sat, 12 Aug 2023 at 08:50, Gang Wu  wrote:

> +1
>
> To Mystic:
> The root cause is that the compiler uses signed char to interpret
> char. The error may disappear if -funsigned-char is added to the
> compiler options. I will take a look to eliminate the UB issue but
> I do not think this is a blocker to the release.
>
>
> Verified checksum and signature.
> All Java and C++ build and test passed on my Mac locally.
> Tool versions: openjdk 1.8.0_322 and clang 14.0.6
>
> BTW, I initially failed with the following error:
> [ERROR] Failed to execute goal
> com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project
> orc-core: Error extracting protoc for version 3.17.3: Unsupported platform:
> protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
> I did the trick from link [1] and it passed.
>
> [1] https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897
>
> Best,
> Gang
>
>
> On Sat, Aug 12, 2023 at 12:56 PM mystic lama 
> wrote:
>
> > I am getting an error when I ran docker test on my local machine like
> > It could be a local issue, but just want to confirm before sending in my
> > vote
> >
> > Error from log towards end of email.
> >
> > Steps I followed
> > 1. Downloaded from
> https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> > 2. After extracting, ran the command from docker folder
> >
> > bash-5.1$ ./run-all.sh local branch-1.9
> > Launching centos7 as 85438
> > Launching debian10 as 85439
> > Launching debian11 as 85440
> > Launching ubuntu20 as 85441
> > Launching ubuntu22 as 85443
> > Launching fedora37 as 85448
> > Launching debian11_jdk=11 as 85454
> > Launching ubuntu22_jdk=11 as 85461
> > Launching ubuntu22_jdk=11_cc=clang as 85468
> > Waiting for 85438
> > Failed tests
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (64
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED centos7
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (19
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED debian10
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (62
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED debian11
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (42
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED debian11_jdk=11
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (58
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (26
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (32
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED ubuntu22
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (35
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (53
> ms)
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
> >  1 FAILED TEST
> > FAILED ubuntu22_jdk=11_cc=clang
> >
> > =
> > Error from log
> > =
> >
> > [--] 6 tests from ConvertColumnReader
> > [ RUN  ] ConvertColumnReader.betweenNumericWithoutOverflows
> > [   OK ] ConvertColumnReader.betweenNumericWithoutOverflows (7 ms)
> > [ RUN  ] ConvertColumnReader.betweenNumricOverflows
> > [   OK ] ConvertColumnReader.betweenNumricOverflows (0 ms)
> > [ RUN  ] ConvertColumnReader.booleanToString
> > [   OK ] ConvertColumnReader.booleanToString (0 ms)
> > [ RUN  ] ConvertColumnReader.TestConvertNumericToStringVariant
> > /root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
> > Expected equality of these values:
> >   expected[1]
> > Which is: "254"
> >   std::string(readC1.data[i], static_cast(readC1.length[i]))
> > Which is: "-2"
> > 2
> > 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-12 Thread Gang Wu
+1

To Mystic:
The root cause is that the compiler uses signed char to interpret
char. The error may disappear if -funsigned-char is added to the
compiler options. I will take a look to eliminate the UB issue but
I do not think this is a blocker to the release.


Verified checksum and signature.
All Java and C++ build and test passed on my Mac locally.
Tool versions: openjdk 1.8.0_322 and clang 14.0.6

BTW, I initially failed with the following error:
[ERROR] Failed to execute goal
com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project
orc-core: Error extracting protoc for version 3.17.3: Unsupported platform:
protoc-3.17.3-osx-aarch_64.exe -> [Help 1]
I did the trick from link [1] and it passed.

[1] https://github.com/os72/protoc-jar/issues/93#issuecomment-1142635897

Best,
Gang


On Sat, Aug 12, 2023 at 12:56 PM mystic lama 
wrote:

> I am getting an error when I ran docker test on my local machine like
> It could be a local issue, but just want to confirm before sending in my
> vote
>
> Error from log towards end of email.
>
> Steps I followed
> 1. Downloaded from https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
> 2. After extracting, ran the command from docker folder
>
> bash-5.1$ ./run-all.sh local branch-1.9
> Launching centos7 as 85438
> Launching debian10 as 85439
> Launching debian11 as 85440
> Launching ubuntu20 as 85441
> Launching ubuntu22 as 85443
> Launching fedora37 as 85448
> Launching debian11_jdk=11 as 85454
> Launching ubuntu22_jdk=11 as 85461
> Launching ubuntu22_jdk=11_cc=clang as 85468
> Waiting for 85438
> Failed tests
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (64 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED centos7
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (19 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED debian10
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (62 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED debian11
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (42 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED debian11_jdk=11
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (58 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (26 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (32 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED ubuntu22
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (35 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (53 ms)
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
>  1 FAILED TEST
> FAILED ubuntu22_jdk=11_cc=clang
>
> =
> Error from log
> =
>
> [--] 6 tests from ConvertColumnReader
> [ RUN  ] ConvertColumnReader.betweenNumericWithoutOverflows
> [   OK ] ConvertColumnReader.betweenNumericWithoutOverflows (7 ms)
> [ RUN  ] ConvertColumnReader.betweenNumricOverflows
> [   OK ] ConvertColumnReader.betweenNumricOverflows (0 ms)
> [ RUN  ] ConvertColumnReader.booleanToString
> [   OK ] ConvertColumnReader.booleanToString (0 ms)
> [ RUN  ] ConvertColumnReader.TestConvertNumericToStringVariant
> /root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
> Expected equality of these values:
>   expected[1]
> Which is: "254"
>   std::string(readC1.data[i], static_cast(readC1.length[i]))
> Which is: "-2"
> 2
> /root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
> Expected equality of these values:
>   expected[1]
> Which is: "252"
>   std::string(readC1.data[i], static_cast(readC1.length[i]))
> Which is: "-4"
> 4
> /root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
> Expected equality of these values:
>   expected[1]
> Which is: "250"
>   std::string(readC1.data[i], static_cast(readC1.length[i]))
> Which is: "-6"
> 6
> /root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
> Expected equality of these values:
>   expected[1]
> Which is: "248"
>   std::string(readC1.data[i], static_cast(readC1.length[i]))
> Which is: "-8"
>
> On Fri, 11 Aug 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-11 Thread mystic lama
I am getting an error when I ran docker test on my local machine like
It could be a local issue, but just want to confirm before sending in my
vote

Error from log towards end of email.

Steps I followed
1. Downloaded from https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0/
2. After extracting, ran the command from docker folder

bash-5.1$ ./run-all.sh local branch-1.9
Launching centos7 as 85438
Launching debian10 as 85439
Launching debian11 as 85440
Launching ubuntu20 as 85441
Launching ubuntu22 as 85443
Launching fedora37 as 85448
Launching debian11_jdk=11 as 85454
Launching ubuntu22_jdk=11 as 85461
Launching ubuntu22_jdk=11_cc=clang as 85468
Waiting for 85438
Failed tests
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (64 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED centos7
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (19 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED debian10
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (62 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED debian11
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (42 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED debian11_jdk=11
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (58 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (26 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (32 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED ubuntu22
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (35 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant (53 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ConvertColumnReader.TestConvertNumericToStringVariant
 1 FAILED TEST
FAILED ubuntu22_jdk=11_cc=clang

=
Error from log
=

[--] 6 tests from ConvertColumnReader
[ RUN  ] ConvertColumnReader.betweenNumericWithoutOverflows
[   OK ] ConvertColumnReader.betweenNumericWithoutOverflows (7 ms)
[ RUN  ] ConvertColumnReader.betweenNumricOverflows
[   OK ] ConvertColumnReader.betweenNumricOverflows (0 ms)
[ RUN  ] ConvertColumnReader.booleanToString
[   OK ] ConvertColumnReader.booleanToString (0 ms)
[ RUN  ] ConvertColumnReader.TestConvertNumericToStringVariant
/root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
Expected equality of these values:
  expected[1]
Which is: "254"
  std::string(readC1.data[i], static_cast(readC1.length[i]))
Which is: "-2"
2
/root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
Expected equality of these values:
  expected[1]
Which is: "252"
  std::string(readC1.data[i], static_cast(readC1.length[i]))
Which is: "-4"
4
/root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
Expected equality of these values:
  expected[1]
Which is: "250"
  std::string(readC1.data[i], static_cast(readC1.length[i]))
Which is: "-6"
6
/root/orc/c++/test/TestConvertColumnReader.cc:339: Failure
Expected equality of these values:
  expected[1]
Which is: "248"
  std::string(readC1.data[i], static_cast(readC1.length[i]))
Which is: "-8"

On Fri, 11 Aug 2023 at 13:01, Dongjoon Hyun  wrote:

> +1 from my side.
>
> - Checksum and signature are verified.
> - Unit tests are verified by GitHub Action on Ubuntu/MacOS/Windows
> - Various OSs are tested via Docker test on CentOS7, Debian10/11,
> Ubuntu20/22 and Fedora37.
> - Integration tests are passed with Apache Spark and Apache Iceberg
>
> You can check the preparation and test result at
>
> https://github.com/apache/orc/issues/1578
> (Release Apache ORC 1.9.1)
>
> Dongjoon
>
> On 2023/08/11 15:43:00 Dongjoon Hyun wrote:
> > Please vote on releasing the following candidate as Apache ORC version
> > 1.9.1. This vote is open until August 16th 1AM (PST) and passes if a
> > majority +1 PMC votes are cast, with a minimum of 3 +1 votes.
> >
> > [ ] +1 Release this package as Apache ORC 1.9.1
> > [ ] -1 Do not release this package because ...
> >
> > TAG:
> > https://github.com/apache/orc/releases/tag/v1.9.1-rc0
> >
> > RELEASE FILES:
> > https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0
> >
> > STAGING REPOSITORY:
> > 

Re: [VOTE] Release Apache ORC 1.9.1 (RC0)

2023-08-11 Thread Dongjoon Hyun
+1 from my side.

- Checksum and signature are verified.
- Unit tests are verified by GitHub Action on Ubuntu/MacOS/Windows
- Various OSs are tested via Docker test on CentOS7, Debian10/11, Ubuntu20/22 
and Fedora37.
- Integration tests are passed with Apache Spark and Apache Iceberg

You can check the preparation and test result at 

https://github.com/apache/orc/issues/1578
(Release Apache ORC 1.9.1)

Dongjoon

On 2023/08/11 15:43:00 Dongjoon Hyun wrote:
> Please vote on releasing the following candidate as Apache ORC version
> 1.9.1. This vote is open until August 16th 1AM (PST) and passes if a
> majority +1 PMC votes are cast, with a minimum of 3 +1 votes.
> 
> [ ] +1 Release this package as Apache ORC 1.9.1
> [ ] -1 Do not release this package because ...
> 
> TAG:
> https://github.com/apache/orc/releases/tag/v1.9.1-rc0
> 
> RELEASE FILES:
> https://dist.apache.org/repos/dist/dev/orc/v1.9.1-rc0
> 
> STAGING REPOSITORY:
> https://repository.apache.org/content/repositories/orgapacheorc-1071
> 
> LIST OF ISSUES:
> https://issues.apache.org/jira/projects/ORC/versions/12353376
> https://github.com/apache/orc/milestone/22?closed=1
> 
> Thanks,
> Dongjoon.
>