You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by GitBox <gi...@apache.org> on 2022/12/07 08:27:55 UTC
[GitHub] [hive] amanraj2520 opened a new pull request, #3841: HIVE-26816 : Add Jenkins file for branch-3
amanraj2520 opened a new pull request, #3841:
URL: https://github.com/apache/hive/pull/3841
HIVE-26816 : Add Jenkins file for branch-3
### What changes were proposed in this pull request?
Added Jenkins file for pull requests made to branch-3.
### Why are the changes needed?
To validate that the changes people are making to branch-3 do not break.
### Does this PR introduce _any_ user-facing change?
No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1343814639
@nrg4878 @sunchao Can you try merging this PR. I do not have access to do that
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1358944002
@zabetak Can you please mark this PR as merged and not closed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1343175475
@ayushtkn @cnauroth Then are we good to approve this? Please let me know if you need anything else. This is blocking cherry picks for 3.2.0 release so was considering merging it by today so that we can start development from tomorrow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on a diff in pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on code in PR #3841:
URL: https://github.com/apache/hive/pull/3841#discussion_r1043699552
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
Review Comment:
Yeah that is true
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ayushtkn commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
ayushtkn commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1343606688
Nothing to add from my side either, Will let people tracking the 3.x release merge it :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] zabetak closed pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
zabetak closed pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
URL: https://github.com/apache/hive/pull/3841
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] zabetak commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
zabetak commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1358949849
@amanraj2520 Nope its not possible to do this. Closed does mean that it is not merged and its a common way of resolving PRs in this and other projects as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] zabetak commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
zabetak commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1358963788
The "Merged" status appears when someone uses the respective UI button in the PR. When a PR is merged via the command line (as I usually do) it appears as "Closed".
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] cnauroth commented on a diff in pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
cnauroth commented on code in PR #3841:
URL: https://github.com/apache/hive/pull/3841#discussion_r1043699725
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
+ string(name: 'OPTS', defaultValue: '', description: 'additional maven opts'),
+ ])
+])
+
+this.prHead = null;
+def checkPrHead() {
+ if(env.CHANGE_ID) {
+ println("checkPrHead - prHead:" + prHead)
+ println("checkPrHead - prHead2:" + pullRequest.head)
+ if (prHead == null) {
+ prHead = pullRequest.head;
+ } else {
+ if(prHead != pullRequest.head) {
+ currentBuild.result = 'ABORTED'
+ error('Found new changes on PR; aborting current build')
+ }
+ }
+ }
+}
+checkPrHead()
+
+def setPrLabel(String prLabel) {
+ if (env.CHANGE_ID) {
+ def mapping=[
+ "SUCCESS":"tests passed",
+ "UNSTABLE":"tests unstable",
+ "FAILURE":"tests failed",
+ "PENDING":"tests pending",
+ ]
+ def newLabels = []
+ for( String l : pullRequest.labels )
+ newLabels.add(l)
+ for( String l : mapping.keySet() )
+ newLabels.remove(mapping[l])
+ newLabels.add(mapping[prLabel])
+ echo ('' +newLabels)
+ pullRequest.labels=newLabels
+ }
+}
+
+setPrLabel("PENDING");
+
+def executorNode(run) {
+ hdbPodTemplate {
+ timeout(time: 24, unit: 'HOURS') {
+ node(POD_LABEL) {
+ container('hdb') {
+ run()
+ }
+ }
+ }
+ }
+}
+
+def buildHive(args) {
+ configFileProvider([configFile(fileId: 'artifactory', variable: 'SETTINGS')]) {
+ withEnv(["MULTIPLIER=$params.MULTIPLIER","M_OPTS=$params.OPTS"]) {
+ sh '''#!/bin/bash -e
+ls -l
+set -x
+. /etc/profile.d/confs.sh
+export USER="`whoami`"
+export MAVEN_OPTS="-Xmx2g"
+export -n HIVE_CONF_DIR
+cp $SETTINGS .git/settings.xml
+OPTS=" -s $PWD/.git/settings.xml -B -Dtest.groups= "
+OPTS+=" -Pitests,qsplits,dist"
Review Comment:
Sorry, this was my mistake, looking at a bad local checkout of branch-3.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ayushtkn commented on a diff in pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
ayushtkn commented on code in PR #3841:
URL: https://github.com/apache/hive/pull/3841#discussion_r1043683282
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
Review Comment:
The number of tests in master will be way more than what are present in branch-3, so shouldn't be very much required here
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] cnauroth commented on a diff in pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
cnauroth commented on code in PR #3841:
URL: https://github.com/apache/hive/pull/3841#discussion_r1043632082
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
+ string(name: 'OPTS', defaultValue: '', description: 'additional maven opts'),
+ ])
+])
+
+this.prHead = null;
+def checkPrHead() {
+ if(env.CHANGE_ID) {
+ println("checkPrHead - prHead:" + prHead)
+ println("checkPrHead - prHead2:" + pullRequest.head)
+ if (prHead == null) {
+ prHead = pullRequest.head;
+ } else {
+ if(prHead != pullRequest.head) {
+ currentBuild.result = 'ABORTED'
+ error('Found new changes on PR; aborting current build')
+ }
+ }
+ }
+}
+checkPrHead()
+
+def setPrLabel(String prLabel) {
+ if (env.CHANGE_ID) {
+ def mapping=[
+ "SUCCESS":"tests passed",
+ "UNSTABLE":"tests unstable",
+ "FAILURE":"tests failed",
+ "PENDING":"tests pending",
+ ]
+ def newLabels = []
+ for( String l : pullRequest.labels )
+ newLabels.add(l)
+ for( String l : mapping.keySet() )
+ newLabels.remove(mapping[l])
+ newLabels.add(mapping[prLabel])
+ echo ('' +newLabels)
+ pullRequest.labels=newLabels
+ }
+}
+
+setPrLabel("PENDING");
+
+def executorNode(run) {
+ hdbPodTemplate {
+ timeout(time: 24, unit: 'HOURS') {
+ node(POD_LABEL) {
+ container('hdb') {
+ run()
+ }
+ }
+ }
+ }
+}
+
+def buildHive(args) {
+ configFileProvider([configFile(fileId: 'artifactory', variable: 'SETTINGS')]) {
+ withEnv(["MULTIPLIER=$params.MULTIPLIER","M_OPTS=$params.OPTS"]) {
+ sh '''#!/bin/bash -e
+ls -l
+set -x
+. /etc/profile.d/confs.sh
+export USER="`whoami`"
+export MAVEN_OPTS="-Xmx2g"
+export -n HIVE_CONF_DIR
+cp $SETTINGS .git/settings.xml
+OPTS=" -s $PWD/.git/settings.xml -B -Dtest.groups= "
+OPTS+=" -Pitests,qsplits,dist"
Review Comment:
The `iceberg` and `errorProne` profiles are present on branch-3. Include them here?
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
Review Comment:
There was a recent change to increase splits to 22 on master. I'm wondering if that's relevant here for branch-3. CC: @ayushtkn .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1358957323
@zabetak Sure no worries. I just had one question, (just a clarification) for all the merged PR's that I have raised for branch-3 let's say https://github.com/apache/hive/pull/3870 I see them as merged. Is it because they were merged after the Jenkins pipeline setup on branch-3.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1358964421
Got it @zabetak makes sense. Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ayushtkn commented on a diff in pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
ayushtkn commented on code in PR #3841:
URL: https://github.com/apache/hive/pull/3841#discussion_r1043686454
##########
Jenkinsfile:
##########
@@ -0,0 +1,219 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+properties([
+ // max 5 build/branch/day
+ rateLimitBuilds(throttle: [count: 5, durationName: 'day', userBoost: true]),
+ // do not run multiple testruns on the same branch
+ disableConcurrentBuilds(),
+ parameters([
+ string(name: 'SPLIT', defaultValue: '20', description: 'Number of buckets to split tests into.'),
+ string(name: 'OPTS', defaultValue: '', description: 'additional maven opts'),
+ ])
+])
+
+this.prHead = null;
+def checkPrHead() {
+ if(env.CHANGE_ID) {
+ println("checkPrHead - prHead:" + prHead)
+ println("checkPrHead - prHead2:" + pullRequest.head)
+ if (prHead == null) {
+ prHead = pullRequest.head;
+ } else {
+ if(prHead != pullRequest.head) {
+ currentBuild.result = 'ABORTED'
+ error('Found new changes on PR; aborting current build')
+ }
+ }
+ }
+}
+checkPrHead()
+
+def setPrLabel(String prLabel) {
+ if (env.CHANGE_ID) {
+ def mapping=[
+ "SUCCESS":"tests passed",
+ "UNSTABLE":"tests unstable",
+ "FAILURE":"tests failed",
+ "PENDING":"tests pending",
+ ]
+ def newLabels = []
+ for( String l : pullRequest.labels )
+ newLabels.add(l)
+ for( String l : mapping.keySet() )
+ newLabels.remove(mapping[l])
+ newLabels.add(mapping[prLabel])
+ echo ('' +newLabels)
+ pullRequest.labels=newLabels
+ }
+}
+
+setPrLabel("PENDING");
+
+def executorNode(run) {
+ hdbPodTemplate {
+ timeout(time: 24, unit: 'HOURS') {
+ node(POD_LABEL) {
+ container('hdb') {
+ run()
+ }
+ }
+ }
+ }
+}
+
+def buildHive(args) {
+ configFileProvider([configFile(fileId: 'artifactory', variable: 'SETTINGS')]) {
+ withEnv(["MULTIPLIER=$params.MULTIPLIER","M_OPTS=$params.OPTS"]) {
+ sh '''#!/bin/bash -e
+ls -l
+set -x
+. /etc/profile.d/confs.sh
+export USER="`whoami`"
+export MAVEN_OPTS="-Xmx2g"
+export -n HIVE_CONF_DIR
+cp $SETTINGS .git/settings.xml
+OPTS=" -s $PWD/.git/settings.xml -B -Dtest.groups= "
+OPTS+=" -Pitests,qsplits,dist"
Review Comment:
Iceberg code was checked in in 4.0-alpha-1 [HIVE-24958](https://issues.apache.org/jira/browse/HIVE-24958)
I couldn't find the code here in branch-3
https://github.com/apache/hive/tree/branch-3
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #3841: HIVE-26816 : Add Jenkins file for branch-3
Posted by GitBox <gi...@apache.org>.
amanraj2520 commented on PR #3841:
URL: https://github.com/apache/hive/pull/3841#issuecomment-1343177457
Also just to add branch-3 is quite similar to branch-3.1 so I think this Jenkins file should be enough.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org