[GitHub] [incubator-kyuubi] yaooqinn opened a new issue #779: Remove obsolete CODE_OF_CONDUCT.md

2021-07-12 Thread GitBox
yaooqinn opened a new issue #779: URL: https://github.com/apache/incubator-kyuubi/issues/779 ### 1. Describe the feature We shall switch to use https://www.apache.org/foundation/policies/conduct.html now ### 2. Motivation ### 3. Describe the solution ###

[GitHub] [incubator-kyuubi] yaooqinn opened a new pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn opened a new pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778 ### _Why are the changes needed?_ Follow https://cwiki.apache.org/confluence/display/INFRA/git+-+.asf.yaml+features to enable `.asf.yaml` with some Github features - desc

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn commented on pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#issuecomment-878418791 cc @WillemJiang @Apache9 @aajisaka @zjffdu @pan3793 @turboFei @ulysses-you @jhx1008 thanks -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-kyuubi] zhaomin1423 commented on pull request #758: [KYUUBI 661] Add UDF system_user

2021-07-12 Thread GitBox
zhaomin1423 commented on pull request #758: URL: https://github.com/apache/incubator-kyuubi/pull/758#issuecomment-878371360 @ulysses-you Should the system_user return the UserGroupInformation.getCurrentUser? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-kyuubi] Yikun commented on a change in pull request #776: Run kyuubi on kubernetes - make basic dockerfile and example

2021-07-12 Thread GitBox
Yikun commented on a change in pull request #776: URL: https://github.com/apache/incubator-kyuubi/pull/776#discussion_r667992957 ## File path: docker/Dockerfile ## @@ -0,0 +1,61 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license a

[GitHub] [incubator-kyuubi] pan3793 commented on a change in pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
pan3793 commented on a change in pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#discussion_r668088689 ## File path: .asf.yaml ## @@ -0,0 +1,51 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreeme

[GitHub] [incubator-kyuubi] ulysses-you closed pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
ulysses-you closed pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777 -- 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: dev

[GitHub] [incubator-kyuubi] ulysses-you opened a new pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
ulysses-you opened a new pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777 ### _Why are the changes needed?_ * fix bug as a blockMgr dir is created with no file at the begining. * simplify code * add test ### _How was this patch tested?_

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#discussion_r668371348 ## File path: tools/kubernetes/spark-block-cleaner/src/test/scala/org.apache.kyuubi.tools/KubernetesSparkBlockCleanerTest.scala ## @@ -0,0 +1,108

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#discussion_r668371688 ## File path: tools/kubernetes/spark-block-cleaner/src/test/scala/org.apache.kyuubi.tools/KubernetesSparkBlockCleanerTest.scala ## @@ -0,0 +1,108

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#discussion_r668374585 ## File path: tools/kubernetes/spark-block-cleaner/src/main/scala/org/apache/kyuubi/tools/KubernetesSparkBlockCleaner.scala ## @@ -54,153 +56,142

[GitHub] [incubator-kyuubi] timothy65535 opened a new pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
timothy65535 opened a new pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780 ### _Why are the changes needed?_ Fix flaky test: Add config to control if cancel invoke interrupt task on engine. ``` The code passed to eventually never returned

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
yaooqinn commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-878723786 cc @ulysses-you -- 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 spec

[GitHub] [incubator-kyuubi] yaooqinn opened a new issue #781: Fix status badges in README to point the current repo

2021-07-12 Thread GitBox
yaooqinn opened a new issue #781: URL: https://github.com/apache/incubator-kyuubi/issues/781 ### 1. Describe the bug ![GitHub top language](https://img.shields.io/github/languages/top/NetEase/kyuubi) [![GitHub release](https://img.shields.io/github/release/NetEase/kyuubi.svg)](ht

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#discussion_r668387169 ## File path: .asf.yaml ## @@ -0,0 +1,51 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreem

[GitHub] [incubator-kyuubi] yaooqinn opened a new issue #782: SparkProcessBuilder fails to generate SPARK_HOME after codebase migration

2021-07-12 Thread GitBox
yaooqinn opened a new issue #782: URL: https://github.com/apache/incubator-kyuubi/issues/782 ### 1. Describe the bug It causes the Github action and Travis CI fail ### 2. Environments 2.1 Versions - Kyuubi - [ ] 1.1.0 - [ ] 1.0.3 - [ ] 0.8.

[GitHub] [incubator-kyuubi] ulysses-you opened a new pull request #783: [KYUUBI #782] SparkProcessBuilder fails to generate SPARK_HOME after codebase migration

2021-07-12 Thread GitBox
ulysses-you opened a new pull request #783: URL: https://github.com/apache/incubator-kyuubi/pull/783 ### _Why are the changes needed?_ Fix failed test ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
yaooqinn commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-878749331 please wait for https://github.com/apache/incubator-kyuubi/pull/783 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-kyuubi] yaooqinn closed issue #782: SparkProcessBuilder fails to generate SPARK_HOME after codebase migration

2021-07-12 Thread GitBox
yaooqinn closed issue #782: URL: https://github.com/apache/incubator-kyuubi/issues/782 -- 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: dev-unsubsc

[GitHub] [incubator-kyuubi] yaooqinn closed pull request #783: [KYUUBI #782] SparkProcessBuilder fails to generate SPARK_HOME after codebase migration

2021-07-12 Thread GitBox
yaooqinn closed pull request #783: URL: https://github.com/apache/incubator-kyuubi/pull/783 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #783: [KYUUBI #782] SparkProcessBuilder fails to generate SPARK_HOME after codebase migration

2021-07-12 Thread GitBox
yaooqinn commented on pull request #783: URL: https://github.com/apache/incubator-kyuubi/pull/783#issuecomment-878757566 thanks, merged to master -- 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

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
yaooqinn commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-878761605 #783 merged, please trigger ci again thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#issuecomment-878762753 How about add some unit tests? -- 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

[GitHub] [incubator-kyuubi] yaooqinn edited a comment on pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn edited a comment on pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#issuecomment-878762753 How about adding some unit tests? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668413526 ## File path: externals/kyuubi-spark-monitor/src/main/scala/org/apache/kyuubi/engine/spark/monitor/entity/KyuubiStatementInfo.scala ## @@ -0,0 +1

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668413699 ## File path: externals/kyuubi-spark-monitor/src/main/scala/org/apache/kyuubi/engine/spark/monitor/entity/KyuubiStatementInfo.scala ## @@ -0,0 +1

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668413957 ## File path: externals/kyuubi-spark-monitor/src/main/scala/org/apache/kyuubi/engine/spark/monitor/entity/KyuubiStatementInfo.scala ## @@ -0,0 +1

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668414546 ## File path: externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/operation/ExecuteStatement.scala ## @@ -156,6 +167,

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668415174 ## File path: externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/operation/ExecuteStatement.scala ## @@ -58,6 +62,12

[GitHub] [incubator-kyuubi] pan3793 commented on a change in pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
pan3793 commented on a change in pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#discussion_r668415183 ## File path: .asf.yaml ## @@ -0,0 +1,54 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreeme

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#discussion_r668416649 ## File path: .asf.yaml ## @@ -0,0 +1,54 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreem

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#discussion_r668416649 ## File path: .asf.yaml ## @@ -0,0 +1,54 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreem

[GitHub] [incubator-kyuubi-website] turboFei opened a new pull request #1: Enable kyuubi.staged.apache.org for staging preview

2021-07-12 Thread GitBox
turboFei opened a new pull request #1: URL: https://github.com/apache/incubator-kyuubi-website/pull/1 -- 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-ma

[GitHub] [incubator-kyuubi-website] turboFei merged pull request #1: Enable kyuubi.staged.apache.org for staging preview

2021-07-12 Thread GitBox
turboFei merged pull request #1: URL: https://github.com/apache/incubator-kyuubi-website/pull/1 -- 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: de

[GitHub] [incubator-kyuubi] yaooqinn closed pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn closed pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #778: [BUILD] Enable .asf.yaml features

2021-07-12 Thread GitBox
yaooqinn commented on pull request #778: URL: https://github.com/apache/incubator-kyuubi/pull/778#issuecomment-878785756 thanks, merged to master -- 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

[GitHub] [incubator-kyuubi] ulysses-you commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
ulysses-you commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-878789072 hi @timothy65535 the change LGTM, please rebase on latest master to trigger test -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-kyuubi] cutiechi commented on issue #774: How to disable ha zk?

2021-07-12 Thread GitBox
cutiechi commented on issue #774: URL: https://github.com/apache/incubator-kyuubi/issues/774#issuecomment-878789315 @pan3793 Ok, thx -- 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

[GitHub] [incubator-kyuubi] cutiechi closed issue #774: How to disable ha zk?

2021-07-12 Thread GitBox
cutiechi closed issue #774: URL: https://github.com/apache/incubator-kyuubi/issues/774 -- 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: dev-unsubsc

[GitHub] [incubator-kyuubi] cutiechi commented on issue #773: How to configure spark executor pod name prefix?

2021-07-12 Thread GitBox
cutiechi commented on issue #773: URL: https://github.com/apache/incubator-kyuubi/issues/773#issuecomment-878789490 @ulysses-you Ok, thx -- 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 spec

[GitHub] [incubator-kyuubi] cutiechi closed issue #773: How to configure spark executor pod name prefix?

2021-07-12 Thread GitBox
cutiechi closed issue #773: URL: https://github.com/apache/incubator-kyuubi/issues/773 -- 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: dev-unsubsc

[GitHub] [incubator-kyuubi-website] turboFei opened a new pull request #2: Refactor Kyuubi HomePage

2021-07-12 Thread GitBox
turboFei opened a new pull request #2: URL: https://github.com/apache/incubator-kyuubi-website/pull/2 -- 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-ma

[GitHub] [incubator-kyuubi] timothy65535 commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-12 Thread GitBox
timothy65535 commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-878794668 Thanks @yaooqinn @ulysses-you -- 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

[GitHub] [incubator-kyuubi-website] turboFei merged pull request #2: Refactor kyuubi website home page.

2021-07-12 Thread GitBox
turboFei merged pull request #2: URL: https://github.com/apache/incubator-kyuubi-website/pull/2 -- 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: de

[GitHub] [incubator-kyuubi] yaooqinn commented on issue #781: Fix status badges in README to point the current repo

2021-07-12 Thread GitBox
yaooqinn commented on issue #781: URL: https://github.com/apache/incubator-kyuubi/issues/781#issuecomment-878799359 cc @wuchunfu, are you interested in this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
yaooqinn commented on pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#issuecomment-878800268 can you fix the ci? -- 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 s

[GitHub] [incubator-kyuubi] yaooqinn commented on a change in pull request #767: Event Tracking: For statement

2021-07-12 Thread GitBox
yaooqinn commented on a change in pull request #767: URL: https://github.com/apache/incubator-kyuubi/pull/767#discussion_r668449269 ## File path: externals/kyuubi-spark-monitor/src/main/scala/org/apache/kyuubi/engine/spark/monitor/entity/KyuubiStatementInfo.scala ## @@ -0,0 +1

[GitHub] [incubator-kyuubi] wuchunfu commented on issue #781: Fix status badges in README to point the current repo

2021-07-12 Thread GitBox
wuchunfu commented on issue #781: URL: https://github.com/apache/incubator-kyuubi/issues/781#issuecomment-878806223 @yaooqinn OK, I'll do the task. -- 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

[GitHub] [incubator-kyuubi-website] ulysses-you opened a new pull request #3: Add *.iml to .gitignore

2021-07-12 Thread GitBox
ulysses-you opened a new pull request #3: URL: https://github.com/apache/incubator-kyuubi-website/pull/3 Make this project friendly for idea. -- 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

[GitHub] [incubator-kyuubi] yaooqinn commented on issue #779: Remove obsolete CODE_OF_CONDUCT.md

2021-07-12 Thread GitBox
yaooqinn commented on issue #779: URL: https://github.com/apache/incubator-kyuubi/issues/779#issuecomment-878815713 cc @zhang1002 -- 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 c

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-12 Thread GitBox
yaooqinn commented on pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#issuecomment-878826788 please also update module name from `Kyuubi Tool Kubernetes Spark Block Cleaner` to `Kyuubi Project Spark Block Cleaner` -- This is an automated message from the Apach

[GitHub] [incubator-kyuubi] ulysses-you opened a new pull request #784: Remove striction in .asf.yaml

2021-07-13 Thread GitBox
ulysses-you opened a new pull request #784: URL: https://github.com/apache/incubator-kyuubi/pull/784 ### _Why are the changes needed?_ Make `dev/merge_kyuubi_pr.py` available. ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughl

[GitHub] [incubator-kyuubi] wuchunfu opened a new pull request #785: [KYUUBI #781] Fix status badges in README to point the current repo

2021-07-13 Thread GitBox
wuchunfu opened a new pull request #785: URL: https://github.com/apache/incubator-kyuubi/pull/785 ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests i

[GitHub] [incubator-kyuubi] ulysses-you merged pull request #784: Remove striction in .asf.yaml

2021-07-13 Thread GitBox
ulysses-you merged pull request #784: URL: https://github.com/apache/incubator-kyuubi/pull/784 -- 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: dev

[GitHub] [incubator-kyuubi] ulysses-you commented on pull request #784: Remove striction in .asf.yaml

2021-07-13 Thread GitBox
ulysses-you commented on pull request #784: URL: https://github.com/apache/incubator-kyuubi/pull/784#issuecomment-878837943 thanks, merging to master -- 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

[GitHub] [incubator-kyuubi] wForget opened a new issue #786: Engine load balancing strategy

2021-07-13 Thread GitBox
wForget opened a new issue #786: URL: https://github.com/apache/incubator-kyuubi/issues/786 ### 1. Describe the feature The last engine is acquired by default. Is there any plan to support other engine load balancing strategies? Such as load balancing according to the current

[GitHub] [incubator-kyuubi] ulysses-you commented on issue #786: Engine load balancing strategy

2021-07-13 Thread GitBox
ulysses-you commented on issue #786: URL: https://github.com/apache/incubator-kyuubi/issues/786#issuecomment-878858349 Good idea, actually we have some discussion about this internal. Maybe we can have something like engine pool to provide the ability of load balancing. And about th

[GitHub] [incubator-kyuubi] yaooqinn commented on issue #786: Engine load balancing strategy

2021-07-13 Thread GitBox
yaooqinn commented on issue #786: URL: https://github.com/apache/incubator-kyuubi/issues/786#issuecomment-878858464 For engine, we'd call it service discovery, not load balancing. The engine discovery logic is better to bind with client requests, a.k.a, JDBC connections, as an engine is s

[GitHub] [incubator-kyuubi] loovia opened a new issue #787: [question] How to connect kyuubi(ha mode) by beeline

2021-07-13 Thread GitBox
loovia opened a new issue #787: URL: https://github.com/apache/incubator-kyuubi/issues/787 I add kyuubi.ha configures in conf/kyuubi-defaults.conf ``` kyuubi.ha.enabled true kyuubi.ha.zookeeper.namespace kyuubi kyuubi.ha.zookeeper.quorum 1.zookeeper.xyz,2.zookeeper.xyz,3.zookeepe

[GitHub] [incubator-kyuubi] wForget commented on issue #787: [question] How to connect the HA mode Kyuubi with Beeline

2021-07-13 Thread GitBox
wForget commented on issue #787: URL: https://github.com/apache/incubator-kyuubi/issues/787#issuecomment-878868040 maybe you need to use quoted string, like: ``` opt/spark/spark-3.1.1-bin-hadoop2.7/bin/beeline -u "jdbc:hive2://1.zookeeper.xyz:2181,2.zookeeper.xyz:2181,3.zookeeper.xyz

[GitHub] [incubator-kyuubi] wuchunfu commented on issue #779: Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
wuchunfu commented on issue #779: URL: https://github.com/apache/incubator-kyuubi/issues/779#issuecomment-878869619 @yaooqinn If @zhang1002 doesn't have time, I can do it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-kyuubi] yaooqinn opened a new pull request #788: [BUILD] Minimize the build of travis

2021-07-13 Thread GitBox
yaooqinn opened a new pull request #788: URL: https://github.com/apache/incubator-kyuubi/pull/788 ### _Why are the changes needed?_ the Travis ci is not very powerful, we low its workload to a test with default profiles and use it as a backup when GA is out of service som

[GitHub] [incubator-kyuubi] yaooqinn commented on issue #779: Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
yaooqinn commented on issue #779: URL: https://github.com/apache/incubator-kyuubi/issues/779#issuecomment-878875983 @wuchunfu 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 specifi

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #785: [KYUUBI #781] Fix status badges in README to point the current repo

2021-07-13 Thread GitBox
yaooqinn commented on pull request #785: URL: https://github.com/apache/incubator-kyuubi/pull/785#issuecomment-878882938 [![](https://tokei.rs/b1/github.com/apache/incubator-kyuubi)](https://github.com/apache/incubator-kyuubi) ![GitHub top language](https://img.shields.io/github/languag

[GitHub] [incubator-kyuubi] wuchunfu opened a new pull request #789: [KYUUBI #779] Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
wuchunfu opened a new pull request #789: URL: https://github.com/apache/incubator-kyuubi/pull/789 ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests i

[GitHub] [incubator-kyuubi] yaooqinn opened a new pull request #790: BUILD] Update PR Merge tool

2021-07-13 Thread GitBox
yaooqinn opened a new pull request #790: URL: https://github.com/apache/incubator-kyuubi/pull/790 ### _Why are the changes needed?_ change GITHUB_API_BASE = "https://api.github.com/repos/NetEase/kyuubi"; to GITHUB_API_BASE = "https://api.github.com/repos/apache/incubato

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #785: [KYUUBI #781] Fix status badges in README to point the current repo

2021-07-13 Thread GitBox
yaooqinn merged pull request #785: URL: https://github.com/apache/incubator-kyuubi/pull/785 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #789: [KYUUBI #779] Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
yaooqinn commented on pull request #789: URL: https://github.com/apache/incubator-kyuubi/pull/789#issuecomment-878920118 please remove the change in README? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [incubator-kyuubi] wuchunfu commented on pull request #789: [KYUUBI #779] Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
wuchunfu commented on pull request #789: URL: https://github.com/apache/incubator-kyuubi/pull/789#issuecomment-878926401 @yaooqinn I have restored the changes in README. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #790: [BUILD] Update PR Merge tool

2021-07-13 Thread GitBox
yaooqinn merged pull request #790: URL: https://github.com/apache/incubator-kyuubi/pull/790 -- 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: dev-un

[GitHub] [incubator-kyuubi] timothy65535 opened a new issue #791: Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
timothy65535 opened a new issue #791: URL: https://github.com/apache/incubator-kyuubi/issues/791 ### 1. Describe the bug Fix point to right repo in PULL_REQUEST_TEMPLATE. ### 2. Environments 2.1 Versions - Kyuubi - [ ] 1.1.0 - [ ] 1.0.3 - [

[GitHub] [incubator-kyuubi] timothy65535 opened a new pull request #792: [KYUUBI 791] Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
timothy65535 opened a new pull request #792: URL: https://github.com/apache/incubator-kyuubi/pull/792 ### _Why are the changes needed?_ Fix point to right repo in PULL_REQUEST_TEMPLATE. ### _How was this patch tested?_ - [ ] Add some test cases that check the cha

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #792: [KYUUBI 791] Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
yaooqinn commented on pull request #792: URL: https://github.com/apache/incubator-kyuubi/pull/792#issuecomment-878953773 docs/community/contributions.md can we update this too? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #789: [KYUUBI #779] Remove obsolete CODE_OF_CONDUCT.md

2021-07-13 Thread GitBox
yaooqinn merged pull request #789: URL: https://github.com/apache/incubator-kyuubi/pull/789 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn opened a new pull request #793: Set protected_branches explicitly in a nonstrict way

2021-07-13 Thread GitBox
yaooqinn opened a new pull request #793: URL: https://github.com/apache/incubator-kyuubi/pull/793 ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if

[GitHub] [incubator-kyuubi] ulysses-you merged pull request #793: Set protected_branches explicitly in a nonstrict way

2021-07-13 Thread GitBox
ulysses-you merged pull request #793: URL: https://github.com/apache/incubator-kyuubi/pull/793 -- 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: dev

[GitHub] [incubator-kyuubi] ulysses-you merged pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-13 Thread GitBox
ulysses-you merged pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780 -- 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: dev

[GitHub] [incubator-kyuubi] ulysses-you commented on pull request #780: [KYUUBI 770] Fix flaky test: Add config to control if cancel invoke interrupt task on engine

2021-07-13 Thread GitBox
ulysses-you commented on pull request #780: URL: https://github.com/apache/incubator-kyuubi/pull/780#issuecomment-879011684 thanks, merging to master for v1.3.0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-kyuubi] ulysses-you closed issue #770: [TEST] Flaky: Add config to control if cancel invoke interrupt task on engine

2021-07-13 Thread GitBox
ulysses-you closed issue #770: URL: https://github.com/apache/incubator-kyuubi/issues/770 -- 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: dev-unsu

[GitHub] [incubator-kyuubi] ulysses-you closed pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-13 Thread GitBox
ulysses-you closed pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777 -- 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: dev

[GitHub] [incubator-kyuubi] ulysses-you opened a new pull request #794: Block cleaner 2

2021-07-13 Thread GitBox
ulysses-you opened a new pull request #794: URL: https://github.com/apache/incubator-kyuubi/pull/794 ### _Why are the changes needed?_ * fix bug as a blockMgr dir is created with no file at the begining. * simplify code * simplify log * add test ### _How was th

[GitHub] [incubator-kyuubi] ulysses-you commented on pull request #777: Refactor Kyuubi kubernetes block cleaner

2021-07-13 Thread GitBox
ulysses-you commented on pull request #777: URL: https://github.com/apache/incubator-kyuubi/pull/777#issuecomment-879039815 see https://github.com/apache/incubator-kyuubi/pull/794 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [incubator-kyuubi] timothy65535 commented on pull request #792: [KYUUBI 791] Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
timothy65535 commented on pull request #792: URL: https://github.com/apache/incubator-kyuubi/pull/792#issuecomment-879079678 > docs/community/contributions.md can we update this too? Yeah. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-kyuubi] yaooqinn opened a new pull request #795: Docker2

2021-07-13 Thread GitBox
yaooqinn opened a new pull request #795: URL: https://github.com/apache/incubator-kyuubi/pull/795 ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if

[GitHub] [incubator-kyuubi] timothy65535 commented on pull request #788: [BUILD] Minimize the build of travis

2021-07-13 Thread GitBox
timothy65535 commented on pull request #788: URL: https://github.com/apache/incubator-kyuubi/pull/788#issuecomment-879142462 It seems that we can remove `.travis.yml` directly, all the jobs of travis have been replaced by github action. -- This is an automated message from the Apache Git

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #788: [BUILD] Minimize the build of travis

2021-07-13 Thread GitBox
yaooqinn merged pull request #788: URL: https://github.com/apache/incubator-kyuubi/pull/788 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #788: [BUILD] Minimize the build of travis

2021-07-13 Thread GitBox
yaooqinn commented on pull request #788: URL: https://github.com/apache/incubator-kyuubi/pull/788#issuecomment-879149444 > It seems that we can remove `.travis.yml` directly, all the jobs of travis have been replaced by Github action. 1. We have experienced several Github action cras

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #792: [KYUUBI #791] Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
yaooqinn merged pull request #792: URL: https://github.com/apache/incubator-kyuubi/pull/792 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #792: [KYUUBI #791] Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
yaooqinn commented on pull request #792: URL: https://github.com/apache/incubator-kyuubi/pull/792#issuecomment-879153520 thanks, merged to master -- 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

[GitHub] [incubator-kyuubi] yaooqinn closed pull request #776: Run kyuubi on kubernetes - make basic dockerfile and example

2021-07-13 Thread GitBox
yaooqinn closed pull request #776: URL: https://github.com/apache/incubator-kyuubi/pull/776 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn closed issue #791: Fix point to right repo in PULL_REQUEST_TEMPLATE

2021-07-13 Thread GitBox
yaooqinn closed issue #791: URL: https://github.com/apache/incubator-kyuubi/issues/791 -- 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: dev-unsubsc

[GitHub] [incubator-kyuubi] yaooqinn closed issue #781: Fix status badges in README to point the current repo

2021-07-13 Thread GitBox
yaooqinn closed issue #781: URL: https://github.com/apache/incubator-kyuubi/issues/781 -- 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: dev-unsubsc

[GitHub] [incubator-kyuubi] pan3793 opened a new pull request #796: [INFRA] Format asf yaml

2021-07-13 Thread GitBox
pan3793 opened a new pull request #796: URL: https://github.com/apache/incubator-kyuubi/pull/796 ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if

[GitHub] [incubator-kyuubi] pan3793 opened a new pull request #797: [INFRA] Disable branches protection

2021-07-13 Thread GitBox
pan3793 opened a new pull request #797: URL: https://github.com/apache/incubator-kyuubi/pull/797 ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if

[GitHub] [incubator-kyuubi] yaooqinn merged pull request #797: [INFRA] Disable branches protection

2021-07-13 Thread GitBox
yaooqinn merged pull request #797: URL: https://github.com/apache/incubator-kyuubi/pull/797 -- 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: dev-un

[GitHub] [incubator-kyuubi] yaooqinn commented on pull request #797: [INFRA] Disable branches protection

2021-07-13 Thread GitBox
yaooqinn commented on pull request #797: URL: https://github.com/apache/incubator-kyuubi/pull/797#issuecomment-879534529 ``` GitHub Protected Branches has set requires signature setting on branch 'master' to 'False' With regards, ASF Infra. ``` Ideally, we shall r

[GitHub] [incubator-kyuubi] ulysses-you commented on pull request #794: Refactor Kyuubi kubernetes block cleaner

2021-07-13 Thread GitBox
ulysses-you commented on pull request #794: URL: https://github.com/apache/incubator-kyuubi/pull/794#issuecomment-879534631 cc @zwangsheng @yaooqinn -- 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

[GitHub] [incubator-kyuubi] yaooqinn opened a new issue #798: Make distribution with --spark-provided as default an only in release.yml of GA

2021-07-13 Thread GitBox
yaooqinn opened a new issue #798: URL: https://github.com/apache/incubator-kyuubi/issues/798 ### 1. Describe the feature ### 2. Motivation ### 3. Describe the solution ### 4. Additional context -- This is an automated message from the Apache Git Se

[GitHub] [incubator-kyuubi] zwangsheng commented on a change in pull request #794: Refactor Kyuubi kubernetes block cleaner

2021-07-13 Thread GitBox
zwangsheng commented on a change in pull request #794: URL: https://github.com/apache/incubator-kyuubi/pull/794#discussion_r669235075 ## File path: tools/spark-block-cleaner/src/main/scala/org/apache/kyuubi/tools/KubernetesSparkBlockCleaner.scala ## @@ -0,0 +1,205 @@ +/* + * L

[GitHub] [incubator-kyuubi] pan3793 closed pull request #796: [DO-NOT-MERGE] [INFRA] Format asf yaml

2021-07-13 Thread GitBox
pan3793 closed pull request #796: URL: https://github.com/apache/incubator-kyuubi/pull/796 -- 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: dev-uns

  1   2   3   4   5   6   7   8   9   10   >