Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Chunwei Lei
> Chunwei, did `prepareVote` fail when you were making rc0? If so, I think
those nexus repos were a byproduct and can be dropped.

Yes. it indeed failed before success.


Best,
Chunwei


On Fri, Jul 24, 2020 at 12:44 PM Francis Chuang 
wrote:

> Actually, I am not sure why there are 6 repositories for 1.22.0 on nexus
> with 3 being closed and released.
>
> Chunwei, did `prepareVote` fail when you were making rc0? If so, I think
> those nexus repos were a byproduct and can be dropped.
>
> Francis
>
> On 24/07/2020 2:40 pm, Francis Chuang wrote:
> > orgapachecalcite-1096 is closed and released, so that's fine. The other
> > open ones are for rcs that did not pass, so I don't think they should be
> > closed and released.
> >
> > There are currently 3 calcite releases in the release folder:
> > https://dist.apache.org/repos/dist/release/calcite/
> >
> > We should just keep the latest 2 releases, so I think you can just use
> > svn to remove 1.22.0. There's the `removeStaleArtifacts` command [1],
> > but I have not tested it myself.
> >
> > Francis
> >
> > [1]
> >
> https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#removing-stale-artifacts
> >
> >
> >
> > On 24/07/2020 2:38 pm, Haisheng Yuan wrote:
> >> Skipped the step of releaseRepository too.
> >> Now finished release.
> >>
> >> There are several closed or open repos in
> >> https://repository.apache.org/#stagingRepositories,
> >> should I manually release all of them?
> >>
> >> On 2020/07/24 04:25:43, Haisheng Yuan  wrote:
> >>> I skipped the step, but now have this error:
> >>> ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist
> >>>
>  Configure project :
> >>> Building Apache Calcite 1.24.0
> >>>
>  Task :releaseRepository
> >>> Initialized stagingRepositoryId orgapachecalcite-1096
> >>>   for repository nexus
> >>> POST request failed. 500: Server Error, body: [errors:[[id:*,
> >>> msg:Unhandled: Missing staging repository: orgapachecalcite-1096
> >>> ]]]
> >>>
>  Task :releaseRepository FAILED
> >>>
> >>> Build calcite FAILURE reason:
> >>>  Execution failed for task ':releaseRepository':
> >>>  io.codearte.gradle.nexus.infra.NexusHttpResponseException:
> >>> 500: Server Error, body: [errors:[[id:*, msg:Unhandled: Missing
> >>> staging repository: orgapachecalcite-1096
> >>>  ]]]
> >>>  at
> >>>
> io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)
>
> >>>
> >>>
> >>>
> >>>
> >>> On 2020/07/24 04:19:14, Haisheng Yuan  wrote:
>  How can I skip this step?
> 
>  ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf
> 
> > Configure project :
>  Building Apache Calcite 1.24.0
> 
> > Task :publishSvnDist FAILED
>  Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
>  https://dist.apache.org/repos/dist
> 
>  Build calcite FAILURE reason:
>   Execution failed for task ':publishSvnDist':
>   Unable to list folders
>  [dev/calcite/apache-calcite-1.24.0-rc0] at
>  https://dist.apache.org/repos/dist: svn: warning: W160013: URL
>  '
> https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0'
>  non-existent in revision 40660
>   svn: E29: Could not list all targets because some
>  targets don't exist
> 
> 
>  I met an error when executing the command. But the command executed
>  some steps successfully before error out. After I fixed the issue
>  and re-execute the command, now I see this error. Because the
>  folders is already removed from dev directory.
> 
>  On 2020/07/24 04:00:14, Haisheng Yuan  wrote:
> > I will help push the artifacts.
> >
> > On 2020/07/24 03:04:05, Francis Chuang 
> > wrote:
> >> I wonder if this is because only PMCs can push to the SVN repo. If
> >> this
> >> is the case, I think we need to nominate a PMC member to push the
> >> artifacts for you.
> >>
> >> On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> >>> Hi, Francis.
> >>>
> >>> I already put those settings in ~/.gradle/gradle.properties:
> >>>
> >>> $cat ~/.gradle/gradle.properties
> >>> asfCommitterId=chunwei
> >>> asfNexusUsername=chunwei
> >>> asfNexusPassword=xxx
> >>>
> >>> asfSvnUsername=chunwei
> >>> asfSvnPassword=xxx
> >>>
> >>> asfGitSourceUsername=chunweilei
> >>> asfGitSourcePassword=xxx
> >>>
> >>> useGpgCmd=true
> >>> signing.gnupg.useLegacyGpg=true
> >>> signing.gnupg.keyName=1F1597F99BCE51A1
> >>>
> >>> asfTestSvnPassword=test
> >>> asfTestSvnUsername=test
> >>> asfTestGitSourceUsername=test
> >>> asfTestGitSourcePassword=test
> >>> asfTestNexusUsername=test
> >>> asfTestNexusPassword=test
> >>>
> >>>
> >>> Best,
> >>> Chunwei
> >>>
> >>>
> >>> On Fri, Jul 24, 2020 at 10:46 AM 

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
I dry run it, it will remove all the artifacts before 1.24.0. I think that is 
not Calcite's policy. At least it should keep 1.23.0.

On 2020/07/24 04:40:50, Francis Chuang  wrote: 
> orgapachecalcite-1096 is closed and released, so that's fine. The other 
> open ones are for rcs that did not pass, so I don't think they should be 
> closed and released.
> 
> There are currently 3 calcite releases in the release folder: 
> https://dist.apache.org/repos/dist/release/calcite/
> 
> We should just keep the latest 2 releases, so I think you can just use 
> svn to remove 1.22.0. There's the `removeStaleArtifacts` command [1], 
> but I have not tested it myself.
> 
> Francis
> 
> [1] 
> https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#removing-stale-artifacts
> 
> 
> On 24/07/2020 2:38 pm, Haisheng Yuan wrote:
> > Skipped the step of releaseRepository too.
> > Now finished release.
> > 
> > There are several closed or open repos in 
> > https://repository.apache.org/#stagingRepositories,
> > should I manually release all of them?
> > 
> > On 2020/07/24 04:25:43, Haisheng Yuan  wrote:
> >> I skipped the step, but now have this error:
> >> ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist
> >>
> >>> Configure project :
> >> Building Apache Calcite 1.24.0
> >>
> >>> Task :releaseRepository
> >> Initialized stagingRepositoryId orgapachecalcite-1096
> >>   for repository nexus
> >> POST request failed. 500: Server Error, body: [errors:[[id:*, 
> >> msg:Unhandled: Missing staging repository: orgapachecalcite-1096
> >> ]]]
> >>
> >>> Task :releaseRepository FAILED
> >>
> >> Build calcite FAILURE reason:
> >>  Execution failed for task ':releaseRepository':
> >>  io.codearte.gradle.nexus.infra.NexusHttpResponseException: 500: 
> >> Server Error, body: [errors:[[id:*, msg:Unhandled: Missing staging 
> >> repository: orgapachecalcite-1096
> >>  ]]]
> >>  at 
> >> io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)
> >>
> >>
> >>
> >> On 2020/07/24 04:19:14, Haisheng Yuan  wrote:
> >>> How can I skip this step?
> >>>
> >>> ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf
> >>>
>  Configure project :
> >>> Building Apache Calcite 1.24.0
> >>>
>  Task :publishSvnDist FAILED
> >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> >>> https://dist.apache.org/repos/dist
> >>>
> >>> Build calcite FAILURE reason:
> >>>  Execution failed for task ':publishSvnDist':
> >>>  Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] 
> >>> at https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
> >>> 'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0'
> >>>  non-existent in revision 40660
> >>>  svn: E29: Could not list all targets because some targets 
> >>> don't exist
> >>>
> >>>
> >>> I met an error when executing the command. But the command executed some 
> >>> steps successfully before error out. After I fixed the issue and 
> >>> re-execute the command, now I see this error. Because the folders is 
> >>> already removed from dev directory.
> >>>
> >>> On 2020/07/24 04:00:14, Haisheng Yuan  wrote:
>  I will help push the artifacts.
> 
>  On 2020/07/24 03:04:05, Francis Chuang  wrote:
> > I wonder if this is because only PMCs can push to the SVN repo. If this
> > is the case, I think we need to nominate a PMC member to push the
> > artifacts for you.
> >
> > On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> >> Hi, Francis.
> >>
> >> I already put those settings in ~/.gradle/gradle.properties:
> >>
> >> $cat ~/.gradle/gradle.properties
> >> asfCommitterId=chunwei
> >> asfNexusUsername=chunwei
> >> asfNexusPassword=xxx
> >>
> >> asfSvnUsername=chunwei
> >> asfSvnPassword=xxx
> >>
> >> asfGitSourceUsername=chunweilei
> >> asfGitSourcePassword=xxx
> >>
> >> useGpgCmd=true
> >> signing.gnupg.useLegacyGpg=true
> >> signing.gnupg.keyName=1F1597F99BCE51A1
> >>
> >> asfTestSvnPassword=test
> >> asfTestSvnUsername=test
> >> asfTestGitSourceUsername=test
> >> asfTestGitSourcePassword=test
> >> asfTestNexusUsername=test
> >> asfTestNexusPassword=test
> >>
> >>
> >> Best,
> >> Chunwei
> >>
> >>
> >> On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> >> 
> >> wrote:
> >>
> >>> You need to include your username and password using the -Pasf* flags.
> >>>
> >>> Here's how I use those flags to publish an avatica release using 
> >>> docker:
> >>> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> >>>
> >>> Francis
> >>>
> >>> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
>  Hi all,
> 
>  I tried to push the release out to mirrors. But I met an error:
> 

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Francis Chuang
Actually, I am not sure why there are 6 repositories for 1.22.0 on nexus 
with 3 being closed and released.


Chunwei, did `prepareVote` fail when you were making rc0? If so, I think 
those nexus repos were a byproduct and can be dropped.


Francis

On 24/07/2020 2:40 pm, Francis Chuang wrote:
orgapachecalcite-1096 is closed and released, so that's fine. The other 
open ones are for rcs that did not pass, so I don't think they should be 
closed and released.


There are currently 3 calcite releases in the release folder: 
https://dist.apache.org/repos/dist/release/calcite/


We should just keep the latest 2 releases, so I think you can just use 
svn to remove 1.22.0. There's the `removeStaleArtifacts` command [1], 
but I have not tested it myself.


Francis

[1] 
https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#removing-stale-artifacts 




On 24/07/2020 2:38 pm, Haisheng Yuan wrote:

Skipped the step of releaseRepository too.
Now finished release.

There are several closed or open repos in 
https://repository.apache.org/#stagingRepositories,

should I manually release all of them?

On 2020/07/24 04:25:43, Haisheng Yuan  wrote:

I skipped the step, but now have this error:
~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist


Configure project :

Building Apache Calcite 1.24.0


Task :releaseRepository

Initialized stagingRepositoryId orgapachecalcite-1096
  for repository nexus
POST request failed. 500: Server Error, body: [errors:[[id:*, 
msg:Unhandled: Missing staging repository: orgapachecalcite-1096

]]]


Task :releaseRepository FAILED


Build calcite FAILURE reason:
 Execution failed for task ':releaseRepository':
 io.codearte.gradle.nexus.infra.NexusHttpResponseException: 
500: Server Error, body: [errors:[[id:*, msg:Unhandled: Missing 
staging repository: orgapachecalcite-1096

 ]]]
 at 
io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52) 





On 2020/07/24 04:19:14, Haisheng Yuan  wrote:

How can I skip this step?

~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist FAILED
Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist


Build calcite FAILURE reason:
 Execution failed for task ':publishSvnDist':
 Unable to list folders 
[dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0' 
non-existent in revision 40660
 svn: E29: Could not list all targets because some 
targets don't exist



I met an error when executing the command. But the command executed 
some steps successfully before error out. After I fixed the issue 
and re-execute the command, now I see this error. Because the 
folders is already removed from dev directory.


On 2020/07/24 04:00:14, Haisheng Yuan  wrote:

I will help push the artifacts.

On 2020/07/24 03:04:05, Francis Chuang  
wrote:
I wonder if this is because only PMCs can push to the SVN repo. If 
this

is the case, I think we need to nominate a PMC member to push the
artifacts for you.

On 24/07/2020 12:58 pm, Chunwei Lei wrote:

Hi, Francis.

I already put those settings in ~/.gradle/gradle.properties:

$cat ~/.gradle/gradle.properties
asfCommitterId=chunwei
asfNexusUsername=chunwei
asfNexusPassword=xxx

asfSvnUsername=chunwei
asfSvnPassword=xxx

asfGitSourceUsername=chunweilei
asfGitSourcePassword=xxx

useGpgCmd=true
signing.gnupg.useLegacyGpg=true
signing.gnupg.keyName=1F1597F99BCE51A1

asfTestSvnPassword=test
asfTestSvnUsername=test
asfTestGitSourceUsername=test
asfTestGitSourcePassword=test
asfTestNexusUsername=test
asfTestNexusPassword=test


Best,
Chunwei


On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 


wrote:

You need to include your username and password using the -Pasf* 
flags.


Here's how I use those flags to publish an avatica release using 
docker:
https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251 



Francis

On 24/07/2020 12:43 pm, Chunwei Lei wrote:

Hi all,

I tried to push the release out to mirrors. But I met an error:

$ ./gradlew publishDist -Prc=0 -Pasf
Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons 
could not be

reused, use --status for details


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist

Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
https://dist.apache.org/repos/dist
Executing svnmucc. root=https://dist.apache.org/repos/dist,
message=Promoting Apache Calcite calcite-1.24.0-rc0 -> 
calcite-1.24.0 to

release area, commands:
mkdir
release/calcite/apache-calcite-1.24.0
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz 

release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz 


cp
40574


Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
Can everyone please not commit to master branch for a moment?
The release manager hasn't open the master branch yet.

On 2020/07/24 04:38:07, Haisheng Yuan  wrote: 
> Skipped the step of releaseRepository too.
> Now finished release.
> 
> There are several closed or open repos in 
> https://repository.apache.org/#stagingRepositories,
> should I manually release all of them?
> 
> On 2020/07/24 04:25:43, Haisheng Yuan  wrote: 
> > I skipped the step, but now have this error:
> > ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist
> > 
> > > Configure project :
> > Building Apache Calcite 1.24.0
> > 
> > > Task :releaseRepository
> > Initialized stagingRepositoryId orgapachecalcite-1096
> >  for repository nexus
> > POST request failed. 500: Server Error, body: [errors:[[id:*, 
> > msg:Unhandled: Missing staging repository: orgapachecalcite-1096
> > ]]]
> > 
> > > Task :releaseRepository FAILED
> > 
> > Build calcite FAILURE reason:
> > Execution failed for task ':releaseRepository':
> > io.codearte.gradle.nexus.infra.NexusHttpResponseException: 500: 
> > Server Error, body: [errors:[[id:*, msg:Unhandled: Missing staging 
> > repository: orgapachecalcite-1096
> > ]]]
> > at 
> > io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)
> > 
> > 
> > 
> > On 2020/07/24 04:19:14, Haisheng Yuan  wrote: 
> > > How can I skip this step?
> > > 
> > > ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf
> > > 
> > > > Configure project :
> > > Building Apache Calcite 1.24.0
> > > 
> > > > Task :publishSvnDist FAILED
> > > Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> > > https://dist.apache.org/repos/dist
> > > 
> > > Build calcite FAILURE reason:
> > > Execution failed for task ':publishSvnDist':
> > > Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> > > https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
> > > 'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0'
> > >  non-existent in revision 40660
> > > svn: E29: Could not list all targets because some targets 
> > > don't exist
> > > 
> > > 
> > > I met an error when executing the command. But the command executed some 
> > > steps successfully before error out. After I fixed the issue and 
> > > re-execute the command, now I see this error. Because the folders is 
> > > already removed from dev directory.
> > > 
> > > On 2020/07/24 04:00:14, Haisheng Yuan  wrote: 
> > > > I will help push the artifacts.
> > > > 
> > > > On 2020/07/24 03:04:05, Francis Chuang  
> > > > wrote: 
> > > > > I wonder if this is because only PMCs can push to the SVN repo. If 
> > > > > this 
> > > > > is the case, I think we need to nominate a PMC member to push the 
> > > > > artifacts for you.
> > > > > 
> > > > > On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> > > > > > Hi, Francis.
> > > > > > 
> > > > > > I already put those settings in ~/.gradle/gradle.properties:
> > > > > > 
> > > > > > $cat ~/.gradle/gradle.properties
> > > > > > asfCommitterId=chunwei
> > > > > > asfNexusUsername=chunwei
> > > > > > asfNexusPassword=xxx
> > > > > > 
> > > > > > asfSvnUsername=chunwei
> > > > > > asfSvnPassword=xxx
> > > > > > 
> > > > > > asfGitSourceUsername=chunweilei
> > > > > > asfGitSourcePassword=xxx
> > > > > > 
> > > > > > useGpgCmd=true
> > > > > > signing.gnupg.useLegacyGpg=true
> > > > > > signing.gnupg.keyName=1F1597F99BCE51A1
> > > > > > 
> > > > > > asfTestSvnPassword=test
> > > > > > asfTestSvnUsername=test
> > > > > > asfTestGitSourceUsername=test
> > > > > > asfTestGitSourcePassword=test
> > > > > > asfTestNexusUsername=test
> > > > > > asfTestNexusPassword=test
> > > > > > 
> > > > > > 
> > > > > > Best,
> > > > > > Chunwei
> > > > > > 
> > > > > > 
> > > > > > On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> > > > > > 
> > > > > > wrote:
> > > > > > 
> > > > > >> You need to include your username and password using the -Pasf* 
> > > > > >> flags.
> > > > > >>
> > > > > >> Here's how I use those flags to publish an avatica release using 
> > > > > >> docker:
> > > > > >> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> > > > > >>
> > > > > >> Francis
> > > > > >>
> > > > > >> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> > > > > >>> Hi all,
> > > > > >>>
> > > > > >>> I tried to push the release out to mirrors. But I met an error:
> > > > > >>>
> > > > > >>> $ ./gradlew publishDist -Prc=0 -Pasf
> > > > > >>> Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could 
> > > > > >>> not be
> > > > > >>> reused, use --status for details
> > > > > >>>
> > > > >  Configure project :
> > > > > >>> Building Apache Calcite 1.24.0
> > > > > >>>
> > > > >  Task :publishSvnDist
> > > > > >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> > > > > >>> https://dist.apache.org/repos/dist
> > > > > >>> Executing svnmucc. 

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Francis Chuang
orgapachecalcite-1096 is closed and released, so that's fine. The other 
open ones are for rcs that did not pass, so I don't think they should be 
closed and released.


There are currently 3 calcite releases in the release folder: 
https://dist.apache.org/repos/dist/release/calcite/


We should just keep the latest 2 releases, so I think you can just use 
svn to remove 1.22.0. There's the `removeStaleArtifacts` command [1], 
but I have not tested it myself.


Francis

[1] 
https://github.com/vlsi/vlsi-release-plugins/tree/master/plugins/stage-vote-release-plugin#removing-stale-artifacts



On 24/07/2020 2:38 pm, Haisheng Yuan wrote:

Skipped the step of releaseRepository too.
Now finished release.

There are several closed or open repos in 
https://repository.apache.org/#stagingRepositories,
should I manually release all of them?

On 2020/07/24 04:25:43, Haisheng Yuan  wrote:

I skipped the step, but now have this error:
~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist


Configure project :

Building Apache Calcite 1.24.0


Task :releaseRepository

Initialized stagingRepositoryId orgapachecalcite-1096
  for repository nexus
POST request failed. 500: Server Error, body: [errors:[[id:*, msg:Unhandled: 
Missing staging repository: orgapachecalcite-1096
]]]


Task :releaseRepository FAILED


Build calcite FAILURE reason:
 Execution failed for task ':releaseRepository':
 io.codearte.gradle.nexus.infra.NexusHttpResponseException: 500: Server 
Error, body: [errors:[[id:*, msg:Unhandled: Missing staging repository: 
orgapachecalcite-1096
 ]]]
 at 
io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)



On 2020/07/24 04:19:14, Haisheng Yuan  wrote:

How can I skip this step?

~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist FAILED

Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist

Build calcite FAILURE reason:
 Execution failed for task ':publishSvnDist':
 Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0' 
non-existent in revision 40660
 svn: E29: Could not list all targets because some targets don't 
exist


I met an error when executing the command. But the command executed some steps 
successfully before error out. After I fixed the issue and re-execute the 
command, now I see this error. Because the folders is already removed from dev 
directory.

On 2020/07/24 04:00:14, Haisheng Yuan  wrote:

I will help push the artifacts.

On 2020/07/24 03:04:05, Francis Chuang  wrote:

I wonder if this is because only PMCs can push to the SVN repo. If this
is the case, I think we need to nominate a PMC member to push the
artifacts for you.

On 24/07/2020 12:58 pm, Chunwei Lei wrote:

Hi, Francis.

I already put those settings in ~/.gradle/gradle.properties:

$cat ~/.gradle/gradle.properties
asfCommitterId=chunwei
asfNexusUsername=chunwei
asfNexusPassword=xxx

asfSvnUsername=chunwei
asfSvnPassword=xxx

asfGitSourceUsername=chunweilei
asfGitSourcePassword=xxx

useGpgCmd=true
signing.gnupg.useLegacyGpg=true
signing.gnupg.keyName=1F1597F99BCE51A1

asfTestSvnPassword=test
asfTestSvnUsername=test
asfTestGitSourceUsername=test
asfTestGitSourcePassword=test
asfTestNexusUsername=test
asfTestNexusPassword=test


Best,
Chunwei


On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
wrote:


You need to include your username and password using the -Pasf* flags.

Here's how I use those flags to publish an avatica release using docker:
https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251

Francis

On 24/07/2020 12:43 pm, Chunwei Lei wrote:

Hi all,

I tried to push the release out to mirrors. But I met an error:

$ ./gradlew publishDist -Prc=0 -Pasf
Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
reused, use --status for details


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist

Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
https://dist.apache.org/repos/dist
Executing svnmucc. root=https://dist.apache.org/repos/dist,
message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
release area, commands:
mkdir
release/calcite/apache-calcite-1.24.0
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
cp
40574


dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc



release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc

cp
40574


dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512



release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512

rm

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
Skipped the step of releaseRepository too.
Now finished release.

There are several closed or open repos in 
https://repository.apache.org/#stagingRepositories,
should I manually release all of them?

On 2020/07/24 04:25:43, Haisheng Yuan  wrote: 
> I skipped the step, but now have this error:
> ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist
> 
> > Configure project :
> Building Apache Calcite 1.24.0
> 
> > Task :releaseRepository
> Initialized stagingRepositoryId orgapachecalcite-1096
>  for repository nexus
> POST request failed. 500: Server Error, body: [errors:[[id:*, msg:Unhandled: 
> Missing staging repository: orgapachecalcite-1096
> ]]]
> 
> > Task :releaseRepository FAILED
> 
> Build calcite FAILURE reason:
> Execution failed for task ':releaseRepository':
> io.codearte.gradle.nexus.infra.NexusHttpResponseException: 500: 
> Server Error, body: [errors:[[id:*, msg:Unhandled: Missing staging 
> repository: orgapachecalcite-1096
> ]]]
> at 
> io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)
> 
> 
> 
> On 2020/07/24 04:19:14, Haisheng Yuan  wrote: 
> > How can I skip this step?
> > 
> > ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf
> > 
> > > Configure project :
> > Building Apache Calcite 1.24.0
> > 
> > > Task :publishSvnDist FAILED
> > Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> > https://dist.apache.org/repos/dist
> > 
> > Build calcite FAILURE reason:
> > Execution failed for task ':publishSvnDist':
> > Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> > https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
> > 'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0' 
> > non-existent in revision 40660
> > svn: E29: Could not list all targets because some targets don't 
> > exist
> > 
> > 
> > I met an error when executing the command. But the command executed some 
> > steps successfully before error out. After I fixed the issue and re-execute 
> > the command, now I see this error. Because the folders is already removed 
> > from dev directory.
> > 
> > On 2020/07/24 04:00:14, Haisheng Yuan  wrote: 
> > > I will help push the artifacts.
> > > 
> > > On 2020/07/24 03:04:05, Francis Chuang  wrote: 
> > > > I wonder if this is because only PMCs can push to the SVN repo. If this 
> > > > is the case, I think we need to nominate a PMC member to push the 
> > > > artifacts for you.
> > > > 
> > > > On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> > > > > Hi, Francis.
> > > > > 
> > > > > I already put those settings in ~/.gradle/gradle.properties:
> > > > > 
> > > > > $cat ~/.gradle/gradle.properties
> > > > > asfCommitterId=chunwei
> > > > > asfNexusUsername=chunwei
> > > > > asfNexusPassword=xxx
> > > > > 
> > > > > asfSvnUsername=chunwei
> > > > > asfSvnPassword=xxx
> > > > > 
> > > > > asfGitSourceUsername=chunweilei
> > > > > asfGitSourcePassword=xxx
> > > > > 
> > > > > useGpgCmd=true
> > > > > signing.gnupg.useLegacyGpg=true
> > > > > signing.gnupg.keyName=1F1597F99BCE51A1
> > > > > 
> > > > > asfTestSvnPassword=test
> > > > > asfTestSvnUsername=test
> > > > > asfTestGitSourceUsername=test
> > > > > asfTestGitSourcePassword=test
> > > > > asfTestNexusUsername=test
> > > > > asfTestNexusPassword=test
> > > > > 
> > > > > 
> > > > > Best,
> > > > > Chunwei
> > > > > 
> > > > > 
> > > > > On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> > > > > 
> > > > > wrote:
> > > > > 
> > > > >> You need to include your username and password using the -Pasf* 
> > > > >> flags.
> > > > >>
> > > > >> Here's how I use those flags to publish an avatica release using 
> > > > >> docker:
> > > > >> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> > > > >>
> > > > >> Francis
> > > > >>
> > > > >> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> > > > >>> Hi all,
> > > > >>>
> > > > >>> I tried to push the release out to mirrors. But I met an error:
> > > > >>>
> > > > >>> $ ./gradlew publishDist -Prc=0 -Pasf
> > > > >>> Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could 
> > > > >>> not be
> > > > >>> reused, use --status for details
> > > > >>>
> > > >  Configure project :
> > > > >>> Building Apache Calcite 1.24.0
> > > > >>>
> > > >  Task :publishSvnDist
> > > > >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> > > > >>> https://dist.apache.org/repos/dist
> > > > >>> Executing svnmucc. root=https://dist.apache.org/repos/dist,
> > > > >>> message=Promoting Apache Calcite calcite-1.24.0-rc0 -> 
> > > > >>> calcite-1.24.0 to
> > > > >>> release area, commands:
> > > > >>> mkdir
> > > > >>> release/calcite/apache-calcite-1.24.0
> > > > >>> cp
> > > > >>> 40574
> > > > >>> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
> > > > >>> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
> > > > >>> cp

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
I skipped the step, but now have this error:
~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf -x publishSvnDist

> Configure project :
Building Apache Calcite 1.24.0

> Task :releaseRepository
Initialized stagingRepositoryId orgapachecalcite-1096
 for repository nexus
POST request failed. 500: Server Error, body: [errors:[[id:*, msg:Unhandled: 
Missing staging repository: orgapachecalcite-1096
]]]

> Task :releaseRepository FAILED

Build calcite FAILURE reason:
Execution failed for task ':releaseRepository':
io.codearte.gradle.nexus.infra.NexusHttpResponseException: 500: Server 
Error, body: [errors:[[id:*, msg:Unhandled: Missing staging repository: 
orgapachecalcite-1096
]]]
at 
io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.sendRequestHandlingErrors(SimplifiedHttpJsonRestClient.groovy:52)



On 2020/07/24 04:19:14, Haisheng Yuan  wrote: 
> How can I skip this step?
> 
> ~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf
> 
> > Configure project :
> Building Apache Calcite 1.24.0
> 
> > Task :publishSvnDist FAILED
> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> https://dist.apache.org/repos/dist
> 
> Build calcite FAILURE reason:
> Execution failed for task ':publishSvnDist':
> Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
> https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
> 'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0' 
> non-existent in revision 40660
> svn: E29: Could not list all targets because some targets don't 
> exist
> 
> 
> I met an error when executing the command. But the command executed some 
> steps successfully before error out. After I fixed the issue and re-execute 
> the command, now I see this error. Because the folders is already removed 
> from dev directory.
> 
> On 2020/07/24 04:00:14, Haisheng Yuan  wrote: 
> > I will help push the artifacts.
> > 
> > On 2020/07/24 03:04:05, Francis Chuang  wrote: 
> > > I wonder if this is because only PMCs can push to the SVN repo. If this 
> > > is the case, I think we need to nominate a PMC member to push the 
> > > artifacts for you.
> > > 
> > > On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> > > > Hi, Francis.
> > > > 
> > > > I already put those settings in ~/.gradle/gradle.properties:
> > > > 
> > > > $cat ~/.gradle/gradle.properties
> > > > asfCommitterId=chunwei
> > > > asfNexusUsername=chunwei
> > > > asfNexusPassword=xxx
> > > > 
> > > > asfSvnUsername=chunwei
> > > > asfSvnPassword=xxx
> > > > 
> > > > asfGitSourceUsername=chunweilei
> > > > asfGitSourcePassword=xxx
> > > > 
> > > > useGpgCmd=true
> > > > signing.gnupg.useLegacyGpg=true
> > > > signing.gnupg.keyName=1F1597F99BCE51A1
> > > > 
> > > > asfTestSvnPassword=test
> > > > asfTestSvnUsername=test
> > > > asfTestGitSourceUsername=test
> > > > asfTestGitSourcePassword=test
> > > > asfTestNexusUsername=test
> > > > asfTestNexusPassword=test
> > > > 
> > > > 
> > > > Best,
> > > > Chunwei
> > > > 
> > > > 
> > > > On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> > > > 
> > > > wrote:
> > > > 
> > > >> You need to include your username and password using the -Pasf* flags.
> > > >>
> > > >> Here's how I use those flags to publish an avatica release using 
> > > >> docker:
> > > >> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> > > >>
> > > >> Francis
> > > >>
> > > >> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> > > >>> Hi all,
> > > >>>
> > > >>> I tried to push the release out to mirrors. But I met an error:
> > > >>>
> > > >>> $ ./gradlew publishDist -Prc=0 -Pasf
> > > >>> Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not 
> > > >>> be
> > > >>> reused, use --status for details
> > > >>>
> > >  Configure project :
> > > >>> Building Apache Calcite 1.24.0
> > > >>>
> > >  Task :publishSvnDist
> > > >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> > > >>> https://dist.apache.org/repos/dist
> > > >>> Executing svnmucc. root=https://dist.apache.org/repos/dist,
> > > >>> message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 
> > > >>> to
> > > >>> release area, commands:
> > > >>> mkdir
> > > >>> release/calcite/apache-calcite-1.24.0
> > > >>> cp
> > > >>> 40574
> > > >>> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
> > > >>> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
> > > >>> cp
> > > >>> 40574
> > > >>>
> > > >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
> > > >>>
> > > >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
> > > >>> cp
> > > >>> 40574
> > > >>>
> > > >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
> > > >>>
> > > >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
> > > >>> rm
> > > >>> dev/calcite/apache-calcite-1.24.0-rc0
> > > >>> svnmucc: E175013: Access 

Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
How can I skip this step?

~/calcite ❯❯❯ ./gradlew publishDist -Prc=0 -Pasf

> Configure project :
Building Apache Calcite 1.24.0

> Task :publishSvnDist FAILED
Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist

Build calcite FAILURE reason:
Execution failed for task ':publishSvnDist':
Unable to list folders [dev/calcite/apache-calcite-1.24.0-rc0] at 
https://dist.apache.org/repos/dist: svn: warning: W160013: URL 
'https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0' 
non-existent in revision 40660
svn: E29: Could not list all targets because some targets don't 
exist


I met an error when executing the command. But the command executed some steps 
successfully before error out. After I fixed the issue and re-execute the 
command, now I see this error. Because the folders is already removed from dev 
directory.

On 2020/07/24 04:00:14, Haisheng Yuan  wrote: 
> I will help push the artifacts.
> 
> On 2020/07/24 03:04:05, Francis Chuang  wrote: 
> > I wonder if this is because only PMCs can push to the SVN repo. If this 
> > is the case, I think we need to nominate a PMC member to push the 
> > artifacts for you.
> > 
> > On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> > > Hi, Francis.
> > > 
> > > I already put those settings in ~/.gradle/gradle.properties:
> > > 
> > > $cat ~/.gradle/gradle.properties
> > > asfCommitterId=chunwei
> > > asfNexusUsername=chunwei
> > > asfNexusPassword=xxx
> > > 
> > > asfSvnUsername=chunwei
> > > asfSvnPassword=xxx
> > > 
> > > asfGitSourceUsername=chunweilei
> > > asfGitSourcePassword=xxx
> > > 
> > > useGpgCmd=true
> > > signing.gnupg.useLegacyGpg=true
> > > signing.gnupg.keyName=1F1597F99BCE51A1
> > > 
> > > asfTestSvnPassword=test
> > > asfTestSvnUsername=test
> > > asfTestGitSourceUsername=test
> > > asfTestGitSourcePassword=test
> > > asfTestNexusUsername=test
> > > asfTestNexusPassword=test
> > > 
> > > 
> > > Best,
> > > Chunwei
> > > 
> > > 
> > > On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> > > wrote:
> > > 
> > >> You need to include your username and password using the -Pasf* flags.
> > >>
> > >> Here's how I use those flags to publish an avatica release using docker:
> > >> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> > >>
> > >> Francis
> > >>
> > >> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> > >>> Hi all,
> > >>>
> > >>> I tried to push the release out to mirrors. But I met an error:
> > >>>
> > >>> $ ./gradlew publishDist -Prc=0 -Pasf
> > >>> Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
> > >>> reused, use --status for details
> > >>>
> >  Configure project :
> > >>> Building Apache Calcite 1.24.0
> > >>>
> >  Task :publishSvnDist
> > >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> > >>> https://dist.apache.org/repos/dist
> > >>> Executing svnmucc. root=https://dist.apache.org/repos/dist,
> > >>> message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
> > >>> release area, commands:
> > >>> mkdir
> > >>> release/calcite/apache-calcite-1.24.0
> > >>> cp
> > >>> 40574
> > >>> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
> > >>> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
> > >>> cp
> > >>> 40574
> > >>>
> > >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
> > >>>
> > >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
> > >>> cp
> > >>> 40574
> > >>>
> > >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
> > >>>
> > >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
> > >>> rm
> > >>> dev/calcite/apache-calcite-1.24.0-rc0
> > >>> svnmucc: E175013: Access to
> > >>> '/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
> > >>> forbidden
> > >>>
> >  Task :publishSvnDist FAILED
> > >>>
> > >>> Build calcite FAILURE reason:
> > >>>   Execution failed for task ':publishSvnDist':
> > >>>   org.gradle.process.internal.ExecException: Process 'command
> > >>> 'svnmucc'' finished with non-zero exit value 1
> > >>>   at
> > >>>
> > >> org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)
> > >>>   at
> > >>>
> > >> org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)
> > >>>   at
> > >>>
> > >> org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)
> > >>>   at
> > >>>
> > >> org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)
> > >>>   at
> > >>> com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
> > >>>   at
> > >>> org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)
> > >>>
> > >>>
> > >>> Any step I may miss?
> > >>>
> > >>>

Re: [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
Yes, I agree.
I will add the details to the JIRA.
Thanks for reminding.

On 2020/07/23 17:43:24, Julian Hyde  wrote: 
> 
> Haisheng,
> 
> > > Julian wrote:
> > > * why is 4032 'breaking'?
> 
> > Haisheng wrote:
> > With that change, the CalcMergeRule won't match PhysicalNode
> > (including EnumerableCalc) in VolcanoPlanner. Perhaps I should
> > elaborate in the release notes.
> 
> Maybe a little more in the release notes. But please add the full details 
> (including link to the email discussion) in CALCITE-4032. In my opinion JIRA 
> cases are a better place for long-term information than release notes or 
> commit messages.
> 
> Julian
> 
> 


Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Haisheng Yuan
I will help push the artifacts.

On 2020/07/24 03:04:05, Francis Chuang  wrote: 
> I wonder if this is because only PMCs can push to the SVN repo. If this 
> is the case, I think we need to nominate a PMC member to push the 
> artifacts for you.
> 
> On 24/07/2020 12:58 pm, Chunwei Lei wrote:
> > Hi, Francis.
> > 
> > I already put those settings in ~/.gradle/gradle.properties:
> > 
> > $cat ~/.gradle/gradle.properties
> > asfCommitterId=chunwei
> > asfNexusUsername=chunwei
> > asfNexusPassword=xxx
> > 
> > asfSvnUsername=chunwei
> > asfSvnPassword=xxx
> > 
> > asfGitSourceUsername=chunweilei
> > asfGitSourcePassword=xxx
> > 
> > useGpgCmd=true
> > signing.gnupg.useLegacyGpg=true
> > signing.gnupg.keyName=1F1597F99BCE51A1
> > 
> > asfTestSvnPassword=test
> > asfTestSvnUsername=test
> > asfTestGitSourceUsername=test
> > asfTestGitSourcePassword=test
> > asfTestNexusUsername=test
> > asfTestNexusPassword=test
> > 
> > 
> > Best,
> > Chunwei
> > 
> > 
> > On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
> > wrote:
> > 
> >> You need to include your username and password using the -Pasf* flags.
> >>
> >> Here's how I use those flags to publish an avatica release using docker:
> >> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
> >>
> >> Francis
> >>
> >> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> >>> Hi all,
> >>>
> >>> I tried to push the release out to mirrors. But I met an error:
> >>>
> >>> $ ./gradlew publishDist -Prc=0 -Pasf
> >>> Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
> >>> reused, use --status for details
> >>>
>  Configure project :
> >>> Building Apache Calcite 1.24.0
> >>>
>  Task :publishSvnDist
> >>> Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> >>> https://dist.apache.org/repos/dist
> >>> Executing svnmucc. root=https://dist.apache.org/repos/dist,
> >>> message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
> >>> release area, commands:
> >>> mkdir
> >>> release/calcite/apache-calcite-1.24.0
> >>> cp
> >>> 40574
> >>> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
> >>> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
> >>> cp
> >>> 40574
> >>>
> >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
> >>>
> >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
> >>> cp
> >>> 40574
> >>>
> >> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
> >>>
> >> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
> >>> rm
> >>> dev/calcite/apache-calcite-1.24.0-rc0
> >>> svnmucc: E175013: Access to
> >>> '/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
> >>> forbidden
> >>>
>  Task :publishSvnDist FAILED
> >>>
> >>> Build calcite FAILURE reason:
> >>>   Execution failed for task ':publishSvnDist':
> >>>   org.gradle.process.internal.ExecException: Process 'command
> >>> 'svnmucc'' finished with non-zero exit value 1
> >>>   at
> >>>
> >> org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)
> >>>   at
> >>>
> >> org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)
> >>>   at
> >>>
> >> org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)
> >>>   at
> >>>
> >> org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)
> >>>   at
> >>> com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
> >>>   at
> >>> org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)
> >>>
> >>>
> >>> Any step I may miss?
> >>>
> >>>
> >>> Best,
> >>> Chunwei
> >>>
> >>>
> >>> On Fri, Jul 24, 2020 at 12:28 AM Chunwei Lei 
> >> wrote:
> >>>
>  Thanks to everyone who has tested the release candidate and given
>  their comments and votes.
> 
>  The tally is as follows.
> 
>  5 binding +1s:
>  Michael Mior
>  Julian Hyde
>  Haisheng Yuan
>  Francis Chuang
>  Danny Chen
> 
>  4 non-binding +1s:
>  Enrico Olivelli
>  Ruleben Q L
>  Rui Wang
>  Chunwei Lei
> 
>  1 binding 0s:
>  Stamatis Zampetakis
> 
>  No  -1s.
> 
>  Therefore I am delighted to announce that the proposal to release
>  Apache Calcite 1.24.0 has passed.
> 
>  Thanks everyone. We’ll now roll the release out to the mirrors.
> 
>  The release notes will be updated afterward according to the feedback.
> 
> 
>  Best,
>  Chunwei
> 
> >>>
> >>
> > 
> 


Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Francis Chuang
I wonder if this is because only PMCs can push to the SVN repo. If this 
is the case, I think we need to nominate a PMC member to push the 
artifacts for you.


On 24/07/2020 12:58 pm, Chunwei Lei wrote:

Hi, Francis.

I already put those settings in ~/.gradle/gradle.properties:

$cat ~/.gradle/gradle.properties
asfCommitterId=chunwei
asfNexusUsername=chunwei
asfNexusPassword=xxx

asfSvnUsername=chunwei
asfSvnPassword=xxx

asfGitSourceUsername=chunweilei
asfGitSourcePassword=xxx

useGpgCmd=true
signing.gnupg.useLegacyGpg=true
signing.gnupg.keyName=1F1597F99BCE51A1

asfTestSvnPassword=test
asfTestSvnUsername=test
asfTestGitSourceUsername=test
asfTestGitSourcePassword=test
asfTestNexusUsername=test
asfTestNexusPassword=test


Best,
Chunwei


On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
wrote:


You need to include your username and password using the -Pasf* flags.

Here's how I use those flags to publish an avatica release using docker:
https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251

Francis

On 24/07/2020 12:43 pm, Chunwei Lei wrote:

Hi all,

I tried to push the release out to mirrors. But I met an error:

$ ./gradlew publishDist -Prc=0 -Pasf
Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
reused, use --status for details


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist

Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
https://dist.apache.org/repos/dist
Executing svnmucc. root=https://dist.apache.org/repos/dist,
message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
release area, commands:
mkdir
release/calcite/apache-calcite-1.24.0
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
cp
40574


dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc



release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc

cp
40574


dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512



release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512

rm
dev/calcite/apache-calcite-1.24.0-rc0
svnmucc: E175013: Access to
'/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
forbidden


Task :publishSvnDist FAILED


Build calcite FAILURE reason:
  Execution failed for task ':publishSvnDist':
  org.gradle.process.internal.ExecException: Process 'command
'svnmucc'' finished with non-zero exit value 1
  at


org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)

  at


org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)

  at


org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)

  at


org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)

  at
com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
  at
org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)


Any step I may miss?


Best,
Chunwei


On Fri, Jul 24, 2020 at 12:28 AM Chunwei Lei 

wrote:



Thanks to everyone who has tested the release candidate and given
their comments and votes.

The tally is as follows.

5 binding +1s:
Michael Mior
Julian Hyde
Haisheng Yuan
Francis Chuang
Danny Chen

4 non-binding +1s:
Enrico Olivelli
Ruleben Q L
Rui Wang
Chunwei Lei

1 binding 0s:
Stamatis Zampetakis

No  -1s.

Therefore I am delighted to announce that the proposal to release
Apache Calcite 1.24.0 has passed.

Thanks everyone. We’ll now roll the release out to the mirrors.

The release notes will be updated afterward according to the feedback.


Best,
Chunwei









Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Chunwei Lei
Hi, Francis.

I already put those settings in ~/.gradle/gradle.properties:

$cat ~/.gradle/gradle.properties
asfCommitterId=chunwei
asfNexusUsername=chunwei
asfNexusPassword=xxx

asfSvnUsername=chunwei
asfSvnPassword=xxx

asfGitSourceUsername=chunweilei
asfGitSourcePassword=xxx

useGpgCmd=true
signing.gnupg.useLegacyGpg=true
signing.gnupg.keyName=1F1597F99BCE51A1

asfTestSvnPassword=test
asfTestSvnUsername=test
asfTestGitSourceUsername=test
asfTestGitSourcePassword=test
asfTestNexusUsername=test
asfTestNexusPassword=test


Best,
Chunwei


On Fri, Jul 24, 2020 at 10:46 AM Francis Chuang 
wrote:

> You need to include your username and password using the -Pasf* flags.
>
> Here's how I use those flags to publish an avatica release using docker:
> https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251
>
> Francis
>
> On 24/07/2020 12:43 pm, Chunwei Lei wrote:
> > Hi all,
> >
> > I tried to push the release out to mirrors. But I met an error:
> >
> > $ ./gradlew publishDist -Prc=0 -Pasf
> > Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
> > reused, use --status for details
> >
> >> Configure project :
> > Building Apache Calcite 1.24.0
> >
> >> Task :publishSvnDist
> > Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
> > https://dist.apache.org/repos/dist
> > Executing svnmucc. root=https://dist.apache.org/repos/dist,
> > message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
> > release area, commands:
> > mkdir
> > release/calcite/apache-calcite-1.24.0
> > cp
> > 40574
> > dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
> > release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
> > cp
> > 40574
> >
> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
> >
> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
> > cp
> > 40574
> >
> dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
> >
> release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
> > rm
> > dev/calcite/apache-calcite-1.24.0-rc0
> > svnmucc: E175013: Access to
> > '/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
> > forbidden
> >
> >> Task :publishSvnDist FAILED
> >
> > Build calcite FAILURE reason:
> >  Execution failed for task ':publishSvnDist':
> >  org.gradle.process.internal.ExecException: Process 'command
> > 'svnmucc'' finished with non-zero exit value 1
> >  at
> >
> org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)
> >  at
> >
> org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)
> >  at
> >
> org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)
> >  at
> >
> org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)
> >  at
> > com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
> >  at
> > org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)
> >
> >
> > Any step I may miss?
> >
> >
> > Best,
> > Chunwei
> >
> >
> > On Fri, Jul 24, 2020 at 12:28 AM Chunwei Lei 
> wrote:
> >
> >> Thanks to everyone who has tested the release candidate and given
> >> their comments and votes.
> >>
> >> The tally is as follows.
> >>
> >> 5 binding +1s:
> >> Michael Mior
> >> Julian Hyde
> >> Haisheng Yuan
> >> Francis Chuang
> >> Danny Chen
> >>
> >> 4 non-binding +1s:
> >> Enrico Olivelli
> >> Ruleben Q L
> >> Rui Wang
> >> Chunwei Lei
> >>
> >> 1 binding 0s:
> >> Stamatis Zampetakis
> >>
> >> No  -1s.
> >>
> >> Therefore I am delighted to announce that the proposal to release
> >> Apache Calcite 1.24.0 has passed.
> >>
> >> Thanks everyone. We’ll now roll the release out to the mirrors.
> >>
> >> The release notes will be updated afterward according to the feedback.
> >>
> >>
> >> Best,
> >> Chunwei
> >>
> >
>


Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Francis Chuang

You need to include your username and password using the -Pasf* flags.

Here's how I use those flags to publish an avatica release using docker: 
https://github.com/apache/calcite-avatica/blob/master/docker.sh#L251


Francis

On 24/07/2020 12:43 pm, Chunwei Lei wrote:

Hi all,

I tried to push the release out to mirrors. But I met an error:

$ ./gradlew publishDist -Prc=0 -Pasf
Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
reused, use --status for details


Configure project :

Building Apache Calcite 1.24.0


Task :publishSvnDist

Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
https://dist.apache.org/repos/dist
Executing svnmucc. root=https://dist.apache.org/repos/dist,
message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
release area, commands:
mkdir
release/calcite/apache-calcite-1.24.0
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
rm
dev/calcite/apache-calcite-1.24.0-rc0
svnmucc: E175013: Access to
'/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
forbidden


Task :publishSvnDist FAILED


Build calcite FAILURE reason:
 Execution failed for task ':publishSvnDist':
 org.gradle.process.internal.ExecException: Process 'command
'svnmucc'' finished with non-zero exit value 1
 at
org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)
 at
org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)
 at
org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)
 at
org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)
 at
com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
 at
org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)


Any step I may miss?


Best,
Chunwei


On Fri, Jul 24, 2020 at 12:28 AM Chunwei Lei  wrote:


Thanks to everyone who has tested the release candidate and given
their comments and votes.

The tally is as follows.

5 binding +1s:
Michael Mior
Julian Hyde
Haisheng Yuan
Francis Chuang
Danny Chen

4 non-binding +1s:
Enrico Olivelli
Ruleben Q L
Rui Wang
Chunwei Lei

1 binding 0s:
Stamatis Zampetakis

No  -1s.

Therefore I am delighted to announce that the proposal to release
Apache Calcite 1.24.0 has passed.

Thanks everyone. We’ll now roll the release out to the mirrors.

The release notes will be updated afterward according to the feedback.


Best,
Chunwei





Re: [RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Chunwei Lei
Hi all,

I tried to push the release out to mirrors. But I met an error:

$ ./gradlew publishDist -Prc=0 -Pasf
Starting a Gradle Daemon, 3 busy and 1 incompatible Daemons could not be
reused, use --status for details

> Configure project :
Building Apache Calcite 1.24.0

> Task :publishSvnDist
Listing SVN folders [dev/calcite/apache-calcite-1.24.0-rc0] at
https://dist.apache.org/repos/dist
Executing svnmucc. root=https://dist.apache.org/repos/dist,
message=Promoting Apache Calcite calcite-1.24.0-rc0 -> calcite-1.24.0 to
release area, commands:
mkdir
release/calcite/apache-calcite-1.24.0
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.asc
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.asc
cp
40574
dev/calcite/apache-calcite-1.24.0-rc0/apache-calcite-1.24.0-src.tar.gz.sha512
release/calcite/apache-calcite-1.24.0/apache-calcite-1.24.0-src.tar.gz.sha512
rm
dev/calcite/apache-calcite-1.24.0-rc0
svnmucc: E175013: Access to
'/repos/dist/!svn/txr/40659-yas/release/calcite/apache-calcite-1.24.0'
forbidden

> Task :publishSvnDist FAILED

Build calcite FAILURE reason:
Execution failed for task ':publishSvnDist':
org.gradle.process.internal.ExecException: Process 'command
'svnmucc'' finished with non-zero exit value 1
at
org.gradle.process.internal.DefaultExecHandle$ExecResultImpl.assertNormalExitValue(DefaultExecHandle.java:417)
at
org.gradle.process.internal.DefaultExecAction.execute(DefaultExecAction.java:38)
at
org.gradle.process.internal.DefaultExecActionFactory.exec(DefaultExecActionFactory.java:156)
at
org.gradle.api.internal.project.DefaultProject.exec(DefaultProject.java:1147)
at
com.github.vlsi.gradle.release.SvnmuccTask.mucc(SvnmuccTask.kt:137)
at
org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)


Any step I may miss?


Best,
Chunwei


On Fri, Jul 24, 2020 at 12:28 AM Chunwei Lei  wrote:

> Thanks to everyone who has tested the release candidate and given
> their comments and votes.
>
> The tally is as follows.
>
> 5 binding +1s:
> Michael Mior
> Julian Hyde
> Haisheng Yuan
> Francis Chuang
> Danny Chen
>
> 4 non-binding +1s:
> Enrico Olivelli
> Ruleben Q L
> Rui Wang
> Chunwei Lei
>
> 1 binding 0s:
> Stamatis Zampetakis
>
> No  -1s.
>
> Therefore I am delighted to announce that the proposal to release
> Apache Calcite 1.24.0 has passed.
>
> Thanks everyone. We’ll now roll the release out to the mirrors.
>
> The release notes will be updated afterward according to the feedback.
>
>
> Best,
> Chunwei
>


Re: Exception:Catch clause is unreachable

2020-07-23 Thread tonytao

Thanks Julian!

I had queried on result metadata,it has 2 columns.
code:
        ResultSet rs = stmt.executeQuery("select * from hdb.t");
        for (int i = 1; i <= rs.getMetaData().getColumnCount(); i++) {
            System.out.println(rs.getMetaData().getColumnName(i));
        }
        while (rs.next()) {
            System.out.println(rs.getObject(2));
        }
output:
ts
speed
10
20

I'll dig in this jdbc driver today to find  where the this empty 
resultset returned.

Thanks again for your help.

Best regards!

On 7/24/20 1:18 AM, Julian Hyde wrote:

I don’t think you need to debug the code, Tony. It’s not your fault, it is a 
bug in the code generation. So, two actions:

1. Please log a bug with a repro case. (It sounds as if the key factor is a 
table with zero columns.)

2. There is a workaround. Add at least one column to your table.

Julian



On Jul 23, 2020, at 2:18 AM, tonytao  wrote:

Thanks again,danny!

It's hard to set the appropriate breakpoint position.:-). I'll check the jdbc 
api whether it returned correct results.


Best regards!

On 7/23/20 4:44 PM, Danny Chan wrote:

In general, Calcite use janino to compile the generated Java string codes into 
real instances during the query execution, you error throws because the code 
generation generates java file with invalid syntax.

Things are not bad now, you can set up a break point in the debugger, and all 
the generated code would be in the directory you have configured, here, you can 
see the error file path is:


File '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 
20:

The generated files names may change each time you debug it, and disappear when 
you stop debugging, so you need to copy the files out and past it into the IDEA 
again to see which file has the Java syntax error.

Best,
Danny Chan
在 2020年7月23日 +0800 PM4:05,tonytao ,写道:

hi Danny,

Thank you for your kindly help.

Attach is the generated code.but I had no idea how this file generated or fix 
the bug.
I'm sorry that I could not receive your reply with my work email account,so I 
replied with my personal account.

errlog:
at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
at 
org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
at 
org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
at 
org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
at 
org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
at 
org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
at 
org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
... 2 more
Caused by: org.codehaus.commons.compiler.CompileException: File 
'/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 20: 
Catch clause is unreachable
at 
org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
at 
org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)


Thanks!

It seems that you got some problem with the generated code, take this [1] to 
debug the code
and to see which line caused the compilation error.

[1] 
https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij

Best,
Danny Chan
在 2020年7月22日 +0800 AM12:17,taojin ,写道:

hi folks,

I has met a compiler error on calcite-core:1.23.0,jdk version is
:openjdk version "11.0.7" 2020-04-14.

Here is the error log,is this a bug?

Exception in thread "main" java.sql.SQLException: Error while executing
SQL "select * from test.t": Error while compiling generated Java code:
public org.apache.calcite.linq4j.Enumerable bind(final
org.apache.calcite.DataContext root) {
   final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
= new org.apache.calcite.linq4j.function.Function1() {
 public org.apache.calcite.linq4j.function.Function0 apply(final
java.sql.ResultSet resultSet) {
   return new org.apache.calcite.linq4j.function.Function0() {
   public Object apply() {
 try {
   return new Object[0];
 } catch (java.sql.SQLException e) {
   throw new 

Re: [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Julian Hyde


Haisheng,

> > Julian wrote:
> > * why is 4032 'breaking'?

> Haisheng wrote:
> With that change, the CalcMergeRule won't match PhysicalNode
> (including EnumerableCalc) in VolcanoPlanner. Perhaps I should
> elaborate in the release notes.

Maybe a little more in the release notes. But please add the full details 
(including link to the email discussion) in CALCITE-4032. In my opinion JIRA 
cases are a better place for long-term information than release notes or commit 
messages.

Julian



Re: [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Julian Hyde
To re-iterate: The release notes must prominently mention the deprecated APIs, 
how they are going to be removed before 1.25, and that people should change 
their code replace them their replacements ASAP. (Especially if they are seeing 
NPEs.)

> On Jul 23, 2020, at 10:05 AM, Julian Hyde  wrote:
> 
> I see that Chunwei has just send out a [RESULT] email announcing that the 
> vote has passed. This is the right decision, but it is a close call.
> 
> https://issues.apache.org/jira/browse/CALCITE-4136 
>  was discovered during 
> the release vote It is a complicated bug. The problem is real, but only 
> occurs in certain circumstances (particular JDKs, loading classes in 
> particular orders). But the problem is caused by our attempt to fix another 
> bug and comply with semantic versioning - i.e. not removing APIs without 
> prior notice.
> 
> Therefore my recommendation is to release 1.24 RC0, include in the release 
> notes the strong recommendation that people immediately replace calls to 
> deprecated APIs, and that we release 1.25 as soon as possible afterwards.
> 
> I’ll start a thread to discuss the release of 1.25.
> 
> Julian
> 
> 
>> On Jul 23, 2020, at 12:21 AM, Chunwei Lei > > wrote:
>> 
>> Thanks to everyone who has tested the release candidate and given their
>> comments.
>> 
>> I have revised the release notes and updated the LICENSE[1]. Regarding
>> CALCITE-4136[2],
>> I have not looked into it yet. But from what I can see, it is not a
>> blocking issue.
>> 
>> 
>> 
>> [1] https://github.com/chunweilei/calcite/commits/branch-1.24-revised 
>> 
>> [2] https://issues.apache.org/jira/browse/CALCITE-4136 
>> 
>> 
>> 
>> Best,
>> Chunwei
>> 
>> 
>> On Thu, Jul 23, 2020 at 9:43 AM Danny Chan > > wrote:
>> 
>>> Thanks Chunwei for taking care of the release ~
>>> 
>>> +1 (binding)
>>> - verified hashes and checksums
>>> - built from sources and run tests (1.8.0_161 on MacOS Mojave)
>>> - run some tests of Flink
>>> 
>>> Best,
>>> Danny Chan
>>> 在 2020年7月22日 +0800 PM11:50,dev@calcite.apache.org 
>>> ,写道:
 
 +1 (non binding)
 - verified hashes and checksums
 - built from sources and run tests (JDK14 on Linux)
 - run tests of HerdDB and some client application
>>> 
> 



Re: Exception:Catch clause is unreachable

2020-07-23 Thread Julian Hyde
I don’t think you need to debug the code, Tony. It’s not your fault, it is a 
bug in the code generation. So, two actions:

1. Please log a bug with a repro case. (It sounds as if the key factor is a 
table with zero columns.)

2. There is a workaround. Add at least one column to your table.

Julian


> On Jul 23, 2020, at 2:18 AM, tonytao  wrote:
> 
> Thanks again,danny!
> 
> It's hard to set the appropriate breakpoint position.:-). I'll check the jdbc 
> api whether it returned correct results.
> 
> 
> Best regards!
> 
> On 7/23/20 4:44 PM, Danny Chan wrote:
>> In general, Calcite use janino to compile the generated Java string codes 
>> into real instances during the query execution, you error throws because the 
>> code generation generates java file with invalid syntax.
>> 
>> Things are not bad now, you can set up a break point in the debugger, and 
>> all the generated code would be in the directory you have configured, here, 
>> you can see the error file path is:
>> 
>>> File '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, 
>>> Column 20:
>> The generated files names may change each time you debug it, and disappear 
>> when you stop debugging, so you need to copy the files out and past it into 
>> the IDEA again to see which file has the Java syntax error.
>> 
>> Best,
>> Danny Chan
>> 在 2020年7月23日 +0800 PM4:05,tonytao ,写道:
>>> hi Danny,
>>> 
>>> Thank you for your kindly help.
>>> 
>>> Attach is the generated code.but I had no idea how this file generated or 
>>> fix the bug.
>>> I'm sorry that I could not receive your reply with my work email account,so 
>>> I replied with my personal account.
>>> 
>>> errlog:
>>>at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
>>>at 
>>> org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
>>>at 
>>> org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
>>>at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
>>>at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
>>>at 
>>> org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
>>>at 
>>> org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
>>>at 
>>> org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
>>>at 
>>> org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
>>>at 
>>> org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
>>>at 
>>> org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
>>>at 
>>> org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
>>>... 2 more
>>> Caused by: org.codehaus.commons.compiler.CompileException: File 
>>> '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 
>>> 20: Catch clause is unreachable
>>>at 
>>> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
>>>at 
>>> org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
>>>at 
>>> org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
>>>at 
>>> org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)
>>> 
>>> 
>>> Thanks!
>>> 
>>> It seems that you got some problem with the generated code, take this [1] 
>>> to debug the code
>>> and to see which line caused the compilation error.
>>> 
>>> [1] 
>>> https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij
>>> 
>>> Best,
>>> Danny Chan
>>> 在 2020年7月22日 +0800 AM12:17,taojin ,写道:
 hi folks,
 
 I has met a compiler error on calcite-core:1.23.0,jdk version is
 :openjdk version "11.0.7" 2020-04-14.
 
 Here is the error log,is this a bug?
 
 Exception in thread "main" java.sql.SQLException: Error while executing
 SQL "select * from test.t": Error while compiling generated Java code:
 public org.apache.calcite.linq4j.Enumerable bind(final
 org.apache.calcite.DataContext root) {
   final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
 = new org.apache.calcite.linq4j.function.Function1() {
 public org.apache.calcite.linq4j.function.Function0 apply(final
 java.sql.ResultSet resultSet) {
   return new org.apache.calcite.linq4j.function.Function0() {
   public Object apply() {
 try {
   return new Object[0];
 } catch (java.sql.SQLException e) {
   throw new RuntimeException(
 e);
 }
   }
 }
   ;
 }
 public Object apply(final Object resultSet) {
   return apply(
 

[DISCUSS] Towards Calcite 1.25.0

2020-07-23 Thread Julian Hyde
The release vote for 1.24 RC0 just passed [1], and it will be released shortly.

We planned that release 1.24 is a transitional release, and 1.25 will follow 
soon afterwards. 1.24 deprecates some APIs that we intend to remove in 1.25. We 
had to make the release so that people can see that the APIs are deprecated, 
and replace them with the new versions.

During the release vote we ran into 4136 [2]. This is a real bug, caused in 
part by the API changes to deprecate APIs (and provide long-term replacements) 
and in part caused by 4079 [3], the real bug that we are trying to solve in 
1.25.

So, it is important that we release 1.25 soon, and that it solves all of these 
problems. I don’t know whether people will perceive 1.24 as a release “lower 
than our usual quality” (even though there are good reasons for it) but 1.25 
needs to be back to the usual quality.

Julian

[1] https://issues.apache.org/jira/browse/CALCITE-4117 


[2] https://issues.apache.org/jira/browse/CALCITE-4136 


[3] https://issues.apache.org/jira/browse/CALCITE-4079 


Re: [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Julian Hyde
I see that Chunwei has just send out a [RESULT] email announcing that the vote 
has passed. This is the right decision, but it is a close call.

https://issues.apache.org/jira/browse/CALCITE-4136 
 was discovered during the 
release vote It is a complicated bug. The problem is real, but only occurs in 
certain circumstances (particular JDKs, loading classes in particular orders). 
But the problem is caused by our attempt to fix another bug and comply with 
semantic versioning - i.e. not removing APIs without prior notice.

Therefore my recommendation is to release 1.24 RC0, include in the release 
notes the strong recommendation that people immediately replace calls to 
deprecated APIs, and that we release 1.25 as soon as possible afterwards.

I’ll start a thread to discuss the release of 1.25.

Julian


> On Jul 23, 2020, at 12:21 AM, Chunwei Lei  wrote:
> 
> Thanks to everyone who has tested the release candidate and given their
> comments.
> 
> I have revised the release notes and updated the LICENSE[1]. Regarding
> CALCITE-4136[2],
> I have not looked into it yet. But from what I can see, it is not a
> blocking issue.
> 
> 
> 
> [1] https://github.com/chunweilei/calcite/commits/branch-1.24-revised
> [2] https://issues.apache.org/jira/browse/CALCITE-4136
> 
> 
> Best,
> Chunwei
> 
> 
> On Thu, Jul 23, 2020 at 9:43 AM Danny Chan  wrote:
> 
>> Thanks Chunwei for taking care of the release ~
>> 
>> +1 (binding)
>> - verified hashes and checksums
>> - built from sources and run tests (1.8.0_161 on MacOS Mojave)
>> - run some tests of Flink
>> 
>> Best,
>> Danny Chan
>> 在 2020年7月22日 +0800 PM11:50,dev@calcite.apache.org,写道:
>>> 
>>> +1 (non binding)
>>> - verified hashes and checksums
>>> - built from sources and run tests (JDK14 on Linux)
>>> - run tests of HerdDB and some client application
>> 



[RESULT] [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Chunwei Lei
Thanks to everyone who has tested the release candidate and given
their comments and votes.

The tally is as follows.

5 binding +1s:
Michael Mior
Julian Hyde
Haisheng Yuan
Francis Chuang
Danny Chen

4 non-binding +1s:
Enrico Olivelli
Ruleben Q L
Rui Wang
Chunwei Lei

1 binding 0s:
Stamatis Zampetakis

No  -1s.

Therefore I am delighted to announce that the proposal to release
Apache Calcite 1.24.0 has passed.

Thanks everyone. We’ll now roll the release out to the mirrors.

The release notes will be updated afterward according to the feedback.


Best,
Chunwei


Re: Exception:Catch clause is unreachable

2020-07-23 Thread tonytao

Thanks again,danny!

It's hard to set the appropriate breakpoint position.:-). I'll check the 
jdbc api whether it returned correct results.



Best regards!

On 7/23/20 4:44 PM, Danny Chan wrote:

In general, Calcite use janino to compile the generated Java string codes into 
real instances during the query execution, you error throws because the code 
generation generates java file with invalid syntax.

Things are not bad now, you can set up a break point in the debugger, and all 
the generated code would be in the directory you have configured, here, you can 
see the error file path is:


File '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 
20:

The generated files names may change each time you debug it, and disappear when 
you stop debugging, so you need to copy the files out and past it into the IDEA 
again to see which file has the Java syntax error.

Best,
Danny Chan
在 2020年7月23日 +0800 PM4:05,tonytao ,写道:

hi Danny,

Thank you for your kindly help.

Attach is the generated code.but I had no idea how this file generated or fix 
the bug.
I'm sorry that I could not receive your reply with my work email account,so I 
replied with my personal account.

errlog:
at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
at 
org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
at 
org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
at 
org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
at 
org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
at 
org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
at 
org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
... 2 more
Caused by: org.codehaus.commons.compiler.CompileException: File 
'/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 20: 
Catch clause is unreachable
at 
org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
at 
org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)


Thanks!

It seems that you got some problem with the generated code, take this [1] to 
debug the code
and to see which line caused the compilation error.

[1] 
https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij

Best,
Danny Chan
在 2020年7月22日 +0800 AM12:17,taojin ,写道:

hi folks,

I has met a compiler error on calcite-core:1.23.0,jdk version is
:openjdk version "11.0.7" 2020-04-14.

Here is the error log,is this a bug?

Exception in thread "main" java.sql.SQLException: Error while executing
SQL "select * from test.t": Error while compiling generated Java code:
public org.apache.calcite.linq4j.Enumerable bind(final
org.apache.calcite.DataContext root) {
   final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
= new org.apache.calcite.linq4j.function.Function1() {
 public org.apache.calcite.linq4j.function.Function0 apply(final
java.sql.ResultSet resultSet) {
   return new org.apache.calcite.linq4j.function.Function0() {
   public Object apply() {
 try {
   return new Object[0];
 } catch (java.sql.SQLException e) {
   throw new RuntimeException(
 e);
 }
   }
 }
   ;
 }
 public Object apply(final Object resultSet) {
   return apply(
 (java.sql.ResultSet) resultSet);
 }
   }
   ;
   final org.apache.calcite.runtime.ResultSetEnumerable enumerable =
org.apache.calcite.runtime.ResultSetEnumerable.of((javax.sql.DataSource)
root.getRootSchema().getSubSchema("test").unwrap(javax.sql.DataSource.class),
"SELECT *\nFROM hdb.t", rowBuilderFactory);
   enumerable.setTimeout(root);
   return enumerable;
}


public Class getElementType() {
   return org.apache.calcite.runtime.FlatLists.ComparableList.class;
}



 at org.apache.calcite.avatica.Helper.createException(Helper.java:56)
 at org.apache.calcite.avatica.Helper.createException(Helper.java:41)
 at
org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:163)
 at

Re: Re: Exception:Catch clause is unreachable

2020-07-23 Thread Feng Zhu
 Hi tony:

I guess there is no column in you table *test.t*.

This piece of code is generated by *JdbcToEnumerableConverter*.
In general, the logic extracts fields from *ResultSet*, and the catch
clause is necessary.
When the field count is 0, we can shot the issue.

try {
>   final Object[] values = new Object[2];
>   values[0] = resultSet.getObject(1);
>   values[1] = resultSet.getDouble(2);
>   .
>   return values;
> } catch (java.sql.SQLException e) {
>   throw new RuntimeException(
> e);
> }


I think it is a bug. We can log it in JIRA.

Bests,
Feng


tonytao  于2020年7月23日周四 下午4:05写道:

> hi Danny,
>
> Thank you for your kindly help.
>
> Attach is the generated code.but I had no idea how this file generated or fix 
> the bug.
> I'm sorry that I could not receive your reply with my work email account,so I 
> replied with my personal account.
>
> errlog:
>   at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
>   at 
> org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
>   at 
> org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
>   at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
>   at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
>   at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
>   at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
>   at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
>   at 
> org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
>   at 
> org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
>   at 
> org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
>   at 
> org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
>   ... 2 more
> Caused by: org.codehaus.commons.compiler.CompileException: File 
> '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 20: 
> Catch clause is unreachable
>   at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
>   at 
> org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
>   at 
> org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
>   at 
> org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)
>
>
> Thanks!
>
> It seems that you got some problem with the generated code, take this [1] to 
> debug the code
> and to see which line caused the compilation error.
>
> [1] 
> https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij
>
> Best,
> Danny Chan
> 在 2020年7月22日 +0800 AM12:17,taojin  ,写道:
> > hi folks,
> >
> > I has met a compiler error on calcite-core:1.23.0,jdk version is
> > :openjdk version "11.0.7" 2020-04-14.
> >
> > Here is the error log,is this a bug?
> >
> > Exception in thread "main" java.sql.SQLException: Error while executing
> > SQL "select * from test.t": Error while compiling generated Java code:
> > public org.apache.calcite.linq4j.Enumerable bind(final
> > org.apache.calcite.DataContext root) {
> >   final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
> > = new org.apache.calcite.linq4j.function.Function1() {
> > public org.apache.calcite.linq4j.function.Function0 apply(final
> > java.sql.ResultSet resultSet) {
> >   return new org.apache.calcite.linq4j.function.Function0() {
> >   public Object apply() {
> > try {
> >   return new Object[0];
> > } catch (java.sql.SQLException e) {
> >   throw new RuntimeException(
> > e);
> > }
> >   }
> > }
> >   ;
> > }
> > public Object apply(final Object resultSet) {
> >   return apply(
> > (java.sql.ResultSet) resultSet);
> > }
> >   }
> >   ;
> >   final org.apache.calcite.runtime.ResultSetEnumerable enumerable =
> > org.apache.calcite.runtime.ResultSetEnumerable.of((javax.sql.DataSource)
> > root.getRootSchema().getSubSchema("test").unwrap(javax.sql.DataSource.class),
> > "SELECT *\nFROM hdb.t", rowBuilderFactory);
> >   enumerable.setTimeout(root);
> >   return enumerable;
> > }
> >
> >
> > public Class getElementType() {
> >   return org.apache.calcite.runtime.FlatLists.ComparableList.class;
> > }
> >
> >
> >
> > at org.apache.calcite.avatica.Helper.createException(Helper.java:56)
> > at org.apache.calcite.avatica.Helper.createException(Helper.java:41)
> > at
> > org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:163)
> > at
> 

Re: Re: Exception:Catch clause is unreachable

2020-07-23 Thread Danny Chan
In general, Calcite use janino to compile the generated Java string codes into 
real instances during the query execution, you error throws because the code 
generation generates java file with invalid syntax.

Things are not bad now, you can set up a break point in the debugger, and all 
the generated code would be in the directory you have configured, here, you can 
see the error file path is:

> File '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, 
> Column 20:

The generated files names may change each time you debug it, and disappear when 
you stop debugging, so you need to copy the files out and past it into the IDEA 
again to see which file has the Java syntax error.

Best,
Danny Chan
在 2020年7月23日 +0800 PM4:05,tonytao ,写道:
> hi Danny,
>
> Thank you for your kindly help.
>
> Attach is the generated code.but I had no idea how this file generated or fix 
> the bug.
> I'm sorry that I could not receive your reply with my work email account,so I 
> replied with my personal account.
>
> errlog:
>at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
>at 
> org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
>at 
> org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
>at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
>at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
>at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
>at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
>at 
> org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
>at 
> org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
>at 
> org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
>at 
> org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
>at 
> org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
>... 2 more
> Caused by: org.codehaus.commons.compiler.CompileException: File 
> '/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 20: 
> Catch clause is unreachable
>at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
>at 
> org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
>at 
> org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
>at 
> org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)
>
>
> Thanks!
>
> It seems that you got some problem with the generated code, take this [1] to 
> debug the code
> and to see which line caused the compilation error.
>
> [1] 
> https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij
>
> Best,
> Danny Chan
> 在 2020年7月22日 +0800 AM12:17,taojin ,写道:
> > hi folks,
> >
> > I has met a compiler error on calcite-core:1.23.0,jdk version is
> > :openjdk version "11.0.7" 2020-04-14.
> >
> > Here is the error log,is this a bug?
> >
> > Exception in thread "main" java.sql.SQLException: Error while executing
> > SQL "select * from test.t": Error while compiling generated Java code:
> > public org.apache.calcite.linq4j.Enumerable bind(final
> > org.apache.calcite.DataContext root) {
> >   final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
> > = new org.apache.calcite.linq4j.function.Function1() {
> > public org.apache.calcite.linq4j.function.Function0 apply(final
> > java.sql.ResultSet resultSet) {
> >   return new org.apache.calcite.linq4j.function.Function0() {
> >   public Object apply() {
> > try {
> >   return new Object[0];
> > } catch (java.sql.SQLException e) {
> >   throw new RuntimeException(
> > e);
> > }
> >   }
> > }
> >   ;
> > }
> > public Object apply(final Object resultSet) {
> >   return apply(
> > (java.sql.ResultSet) resultSet);
> > }
> >   }
> >   ;
> >   final org.apache.calcite.runtime.ResultSetEnumerable enumerable =
> > org.apache.calcite.runtime.ResultSetEnumerable.of((javax.sql.DataSource)
> > root.getRootSchema().getSubSchema("test").unwrap(javax.sql.DataSource.class),
> > "SELECT *\nFROM hdb.t", rowBuilderFactory);
> >   enumerable.setTimeout(root);
> >   return enumerable;
> > }
> >
> >
> > public Class getElementType() {
> >   return org.apache.calcite.runtime.FlatLists.ComparableList.class;
> > }
> >
> >
> >
> > at org.apache.calcite.avatica.Helper.createException(Helper.java:56)
> > at org.apache.calcite.avatica.Helper.createException(Helper.java:41)
> > at
> > 

Re: Re: Exception:Catch clause is unreachable

2020-07-23 Thread tonytao

hi Danny,

Thank you for your kindly help.

Attach is the generated code.but I had no idea how this file generated or fix 
the bug.
I'm sorry that I could not receive your reply with my work email account,so I 
replied with my personal account.

errlog:
at org.apache.calcite.avatica.Helper.wrap(Helper.java:37)
at 
org.apache.calcite.adapter.enumerable.EnumerableInterpretable.toBindable(EnumerableInterpretable.java:128)
at 
org.apache.calcite.prepare.CalcitePrepareImpl$CalcitePreparingStmt.implement(CalcitePrepareImpl.java:)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:309)
at org.apache.calcite.prepare.Prepare.prepareSql(Prepare.java:208)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare2_(CalcitePrepareImpl.java:632)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepare_(CalcitePrepareImpl.java:498)
at 
org.apache.calcite.prepare.CalcitePrepareImpl.prepareSql(CalcitePrepareImpl.java:468)
at 
org.apache.calcite.jdbc.CalciteConnectionImpl.parseQuery(CalciteConnectionImpl.java:231)
at 
org.apache.calcite.jdbc.CalciteMetaImpl.prepareAndExecute(CalciteMetaImpl.java:552)
at 
org.apache.calcite.avatica.AvaticaConnection.prepareAndExecuteInternal(AvaticaConnection.java:675)
at 
org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:156)
... 2 more
Caused by: org.codehaus.commons.compiler.CompileException: File 
'/home/tony/workspace/tmp/janino7222401528383593043.java', Line 8, Column 20: 
Catch clause is unreachable
at 
org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:12211)
at 
org.codehaus.janino.UnitCompiler.compileTryCatch(UnitCompiler.java:3110)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinally(UnitCompiler.java:2966)
at 
org.codehaus.janino.UnitCompiler.compileTryCatchFinallyWithResources(UnitCompiler.java:2770)


Thanks!

It seems that you got some problem with the generated code, take this [1] to 
debug the code
and to see which line caused the compilation error.

[1] 
https://calcite.apache.org/docs/howto.html#debugging-generated-classes-in-intellij

Best,
Danny Chan
在 2020年7月22日 +0800 AM12:17,taojin ,写道:

hi folks,

I has met a compiler error on calcite-core:1.23.0,jdk version is
:openjdk version "11.0.7" 2020-04-14.

Here is the error log,is this a bug?

Exception in thread "main" java.sql.SQLException: Error while executing
SQL "select * from test.t": Error while compiling generated Java code:
public org.apache.calcite.linq4j.Enumerable bind(final
org.apache.calcite.DataContext root) {
  final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
= new org.apache.calcite.linq4j.function.Function1() {
    public org.apache.calcite.linq4j.function.Function0 apply(final
java.sql.ResultSet resultSet) {
  return new org.apache.calcite.linq4j.function.Function0() {
  public Object apply() {
    try {
  return new Object[0];
    } catch (java.sql.SQLException e) {
  throw new RuntimeException(
    e);
    }
  }
    }
  ;
    }
    public Object apply(final Object resultSet) {
  return apply(
    (java.sql.ResultSet) resultSet);
    }
  }
  ;
  final org.apache.calcite.runtime.ResultSetEnumerable enumerable =
org.apache.calcite.runtime.ResultSetEnumerable.of((javax.sql.DataSource)
root.getRootSchema().getSubSchema("test").unwrap(javax.sql.DataSource.class),
"SELECT *\nFROM hdb.t", rowBuilderFactory);
  enumerable.setTimeout(root);
  return enumerable;
}


public Class getElementType() {
  return org.apache.calcite.runtime.FlatLists.ComparableList.class;
}



    at org.apache.calcite.avatica.Helper.createException(Helper.java:56)
    at org.apache.calcite.avatica.Helper.createException(Helper.java:41)
    at
org.apache.calcite.avatica.AvaticaStatement.executeInternal(AvaticaStatement.java:163)
    at
org.apache.calcite.avatica.AvaticaStatement.executeQuery(AvaticaStatement.java:227)
    at mtest.App5.main(App5.java:45)
Caused by: java.lang.RuntimeException: Error while compiling generated
Java code:
public org.apache.calcite.linq4j.Enumerable bind(final
org.apache.calcite.DataContext root) {
  final org.apache.calcite.linq4j.function.Function1 rowBuilderFactory
= new org.apache.calcite.linq4j.function.Function1() {
    public org.apache.calcite.linq4j.function.Function0 apply(final
java.sql.ResultSet resultSet) {
  return new org.apache.calcite.linq4j.function.Function0() {
  public Object apply() {
    try {
  return new Object[0];
    } catch (java.sql.SQLException e) {
  throw new RuntimeException(
    e);
    }
  }
    }
  ;
    }
    public Object apply(final Object resultSet) {
  return apply(
    (java.sql.ResultSet) resultSet);
    }
  }
  ;
  final 

Re: [VOTE] Release apache-calcite-1.24.0 (release candidate 0)

2020-07-23 Thread Chunwei Lei
Thanks to everyone who has tested the release candidate and given their
comments.

I have revised the release notes and updated the LICENSE[1]. Regarding
CALCITE-4136[2],
I have not looked into it yet. But from what I can see, it is not a
blocking issue.



[1] https://github.com/chunweilei/calcite/commits/branch-1.24-revised
[2] https://issues.apache.org/jira/browse/CALCITE-4136


Best,
Chunwei


On Thu, Jul 23, 2020 at 9:43 AM Danny Chan  wrote:

> Thanks Chunwei for taking care of the release ~
>
> +1 (binding)
> - verified hashes and checksums
> - built from sources and run tests (1.8.0_161 on MacOS Mojave)
> - run some tests of Flink
>
> Best,
> Danny Chan
> 在 2020年7月22日 +0800 PM11:50,dev@calcite.apache.org,写道:
> >
> > +1 (non binding)
> > - verified hashes and checksums
> > - built from sources and run tests (JDK14 on Linux)
> > - run tests of HerdDB and some client application
>