yaooqinn closed issue #779:
URL: https://github.com/apache/incubator-kyuubi/issues/779
--
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
ulysses-you merged pull request #794:
URL: 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 GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev
ulysses-you commented on pull request #794:
URL: https://github.com/apache/incubator-kyuubi/pull/794#issuecomment-879561476
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
yaooqinn merged pull request #799:
URL: https://github.com/apache/incubator-kyuubi/pull/799
--
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
pan3793 merged pull request #800:
URL: https://github.com/apache/incubator-kyuubi/pull/800
--
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
yaooqinn opened a new pull request #800:
URL: https://github.com/apache/incubator-kyuubi/pull/800
Reverts apache/incubator-kyuubi#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
pan3793 opened a new pull request #799:
URL: https://github.com/apache/incubator-kyuubi/pull/799
### _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
yaooqinn merged 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-un
zwangsheng commented on a change in pull request #794:
URL: https://github.com/apache/incubator-kyuubi/pull/794#discussion_r669244265
##
File path:
tools/spark-block-cleaner/src/main/scala/org/apache/kyuubi/tools/KubernetesSparkBlockCleaner.scala
##
@@ -0,0 +1,205 @@
+/*
+ * L
ulysses-you commented on a change in pull request #794:
URL: https://github.com/apache/incubator-kyuubi/pull/794#discussion_r669242136
##
File path:
tools/spark-block-cleaner/src/main/scala/org/apache/kyuubi/tools/KubernetesSparkBlockCleaner.scala
##
@@ -0,0 +1,205 @@
+/*
+ *
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
- [
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
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
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
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
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
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
yaooqinn commented on pull request #785:
URL: https://github.com/apache/incubator-kyuubi/pull/785#issuecomment-878882938
[](https://github.com/apache/incubator-kyuubi)
![GitHub top
language](https://img.shields.io/github/languag
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
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
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
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
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
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
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
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
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
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
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
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
59 matches
Mail list logo
