hit-lacus commented on pull request #1813:
URL: https://github.com/apache/kylin/pull/1813#issuecomment-1033315596
Merged
--
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
hit-lacus merged pull request #1813:
URL: https://github.com/apache/kylin/pull/1813
--
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: issues-unsubsc
hit-lacus merged pull request #1805:
URL: https://github.com/apache/kylin/pull/1805
--
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: issues-unsubsc
hit-lacus commented on pull request #1805:
URL: https://github.com/apache/kylin/pull/1805#issuecomment-1033310204
Thx, @zhouyao1994
--
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
zhangayqian commented on pull request #1813:
URL: https://github.com/apache/kylin/pull/1813#issuecomment-1033245995
@juntaozhang Thanks! LGTM
--
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
pjfanning opened a new pull request #1815:
URL: https://github.com/apache/kylin/pull/1815
## Proposed changes
jackson-databind 2.6.7.5 is a security fix - other jackson jars were not
released with this version number but jackson-databind 2.6.7.5 is comptible
with 2.6.7.
http
codecov-commenter commented on pull request #1814:
URL: https://github.com/apache/kylin/pull/1814#issuecomment-1033135171
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1814?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
pjfanning opened a new pull request #1814:
URL: https://github.com/apache/kylin/pull/1814
## Proposed changes
Try to fix some old dependencies that have known security vulnerabilities
## Types of changes
What types of changes does your code introduce to Kylin?
_Put an
juntaozhang commented on pull request #1813:
URL: https://github.com/apache/kylin/pull/1813#issuecomment-1032571169
https://github.com/apache/kylin/pull/1809
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
juntaozhang closed pull request #1809:
URL: https://github.com/apache/kylin/pull/1809
--
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: issues-unsub
juntaozhang opened a new pull request #1813:
URL: https://github.com/apache/kylin/pull/1813
## Proposed changes
fix[ KYLIN-4382](https://issues.apache.org/jira/browse/KYLIN-4382) Unable to
use DATE type in prepared statements
## Github Branch
As most of the development
zhangayqian commented on pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#issuecomment-1032474330
> > Hi @juntaozhang, I tested this PR, which can fix the bug on kylin3, but
it doesn't work on kylin4. I think this PR should be merged into `kylin3`
branch.
>
> Thank
juntaozhang commented on pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#issuecomment-1032469935
> Hi @juntaozhang, I tested this PR, which can fix the bug on kylin3, but it
doesn't work on kylin4. I think this PR should be merged into `kylin3` branch.
Thanks for y
zhangayqian commented on a change in pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#discussion_r801496762
##
File path: jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java
##
@@ -286,7 +286,7 @@ public void testKYLIN4382() throws SQLException,
ParseExc
zhangayqian commented on pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#issuecomment-1032462697
Hi @juntaozhang, I tested this PR, which can fix the bug on kylin3, but it
doesn't work on kylin4. I think this PR should be merged into `kylin3` branch.
--
This is an auto
Mukvin opened a new pull request #1812:
URL: https://github.com/apache/kylin/pull/1812
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
zhangayqian opened a new pull request #1811:
URL: https://github.com/apache/kylin/pull/1811
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus commented on a change in pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#discussion_r801241687
##
File path: query/src/main/java/org/apache/kylin/query/relnode/OLAPContext.java
##
@@ -337,7 +340,14 @@ private String
transferDateTimeColumnToMillis(C
hit-lacus merged pull request #1810:
URL: https://github.com/apache/kylin/pull/1810
--
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: issues-unsubsc
lgtm-com[bot] commented on pull request #1810:
URL: https://github.com/apache/kylin/pull/1810#issuecomment-1031386945
This pull request **introduces 3 alerts** when merging
be5e2b9da50f8dd387895f5ea7a99ddb753f6341 into
7eafdfffe30239a73aaae28db60e673eb9b94831 - [view on
LGTM.com](https://
Mukvin opened a new pull request #1810:
URL: https://github.com/apache/kylin/pull/1810
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
codecov-commenter edited a comment on pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#issuecomment-1031085519
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1809?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #1809:
URL: https://github.com/apache/kylin/pull/1809#issuecomment-1031085519
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1809?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
juntaozhang opened a new pull request #1809:
URL: https://github.com/apache/kylin/pull/1809
## Proposed changes
fix[ KYLIN-4382](https://issues.apache.org/jira/browse/KYLIN-4382) Unable to
use DATE type in prepared statements
## Github Branch
As most of the development
juntaozhang closed pull request #1808:
URL: https://github.com/apache/kylin/pull/1808
--
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: issues-unsub
juntaozhang opened a new pull request #1808:
URL: https://github.com/apache/kylin/pull/1808
## Proposed changes
fix[ KYLIN-4382](https://issues.apache.org/jira/browse/KYLIN-4382) Unable to
use DATE type in prepared statements
## Github Branch
As most of the development
hit-lacus merged pull request #1807:
URL: https://github.com/apache/kylin/pull/1807
--
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: issues-unsubsc
Mukvin opened a new pull request #1807:
URL: https://github.com/apache/kylin/pull/1807
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
hit-lacus merged pull request #1806:
URL: https://github.com/apache/kylin/pull/1806
--
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: issues-unsubsc
Mukvin opened a new pull request #1806:
URL: https://github.com/apache/kylin/pull/1806
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
codecov-commenter edited a comment on pull request #1805:
URL: https://github.com/apache/kylin/pull/1805#issuecomment-1022181850
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1805?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #1805:
URL: https://github.com/apache/kylin/pull/1805#issuecomment-1022181850
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1805?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
zhouyao1994 opened a new pull request #1805:
URL: https://github.com/apache/kylin/pull/1805
## Proposed changes
update document
## Github Branch
As most of the development works are on Kylin 4, we need to switch it as
main branch. Apache Kylin community changes the bra
hit-lacus merged pull request #1803:
URL: https://github.com/apache/kylin/pull/1803
--
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: issues-unsubsc
hit-lacus merged pull request #1804:
URL: https://github.com/apache/kylin/pull/1804
--
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: issues-unsubsc
hit-lacus merged pull request #1802:
URL: https://github.com/apache/kylin/pull/1802
--
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: issues-unsubsc
hit-lacus merged pull request #1804:
URL: https://github.com/apache/kylin/pull/1804
--
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: issues-unsubsc
Mukvin opened a new pull request #1804:
URL: https://github.com/apache/kylin/pull/1804
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
hit-lacus merged pull request #1803:
URL: https://github.com/apache/kylin/pull/1803
--
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: issues-unsubsc
Mukvin opened a new pull request #1803:
URL: https://github.com/apache/kylin/pull/1803
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
hit-lacus merged pull request #1802:
URL: https://github.com/apache/kylin/pull/1802
--
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: issues-unsubsc
Mukvin opened a new pull request #1802:
URL: https://github.com/apache/kylin/pull/1802
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
hit-lacus merged pull request #1801:
URL: https://github.com/apache/kylin/pull/1801
--
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: issues-unsubsc
Mukvin opened a new pull request #1801:
URL: https://github.com/apache/kylin/pull/1801
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be sure
hit-lacus merged pull request #1798:
URL: https://github.com/apache/kylin/pull/1798
--
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: issues-unsubsc
sleep1661 commented on pull request #1800:
URL: https://github.com/apache/kylin/pull/1800#issuecomment-1017126242
@hit-lacus @zhangayqian @zhengshengjun could you help take a look when you
have time? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the m
sleep1661 opened a new pull request #1800:
URL: https://github.com/apache/kylin/pull/1800
## Proposed changes
Change Volatile Range be 7 by default, which keep hot data of last 7 day not
been merged.
## Github Branch
As most of the development works are on Kylin 4, we
zhangayqian merged pull request #1799:
URL: https://github.com/apache/kylin/pull/1799
--
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: issues-unsub
zhangayqian opened a new pull request #1799:
URL: https://github.com/apache/kylin/pull/1799
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian opened a new pull request #1798:
URL: https://github.com/apache/kylin/pull/1798
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus closed pull request #1793:
URL: https://github.com/apache/kylin/pull/1793
--
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: issues-unsubsc
hit-lacus merged pull request #1797:
URL: https://github.com/apache/kylin/pull/1797
--
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: issues-unsubsc
hit-lacus merged pull request #1767:
URL: https://github.com/apache/kylin/pull/1767
--
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: issues-unsubsc
hit-lacus merged pull request #1770:
URL: https://github.com/apache/kylin/pull/1770
--
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: issues-unsubsc
hit-lacus merged pull request #1791:
URL: https://github.com/apache/kylin/pull/1791
--
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: issues-unsubsc
zhangayqian opened a new pull request #1797:
URL: https://github.com/apache/kylin/pull/1797
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian commented on pull request #1791:
URL: https://github.com/apache/kylin/pull/1791#issuecomment-1012805238
LGTM
--
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
zhangayqian commented on pull request #1790:
URL: https://github.com/apache/kylin/pull/1790#issuecomment-1012796705
LGTM
--
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
zhangayqian commented on pull request #1780:
URL: https://github.com/apache/kylin/pull/1780#issuecomment-1012796343
Hi @fengpod
Similar to https://github.com/apache/kylin/pull/1778, after building a
global dictionary, the dictionary lock is not purged, resulting in the failure
to obtai
zhangayqian commented on a change in pull request #1780:
URL: https://github.com/apache/kylin/pull/1780#discussion_r784476015
##
File path:
source-hive/src/main/java/org/apache/kylin/source/hive/SparkHiveDictReplaceStep.java
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache
zhangayqian commented on a change in pull request #1780:
URL: https://github.com/apache/kylin/pull/1780#discussion_r784476015
##
File path:
source-hive/src/main/java/org/apache/kylin/source/hive/SparkHiveDictReplaceStep.java
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache
zhangayqian opened a new pull request #1796:
URL: https://github.com/apache/kylin/pull/1796
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian merged pull request #1796:
URL: https://github.com/apache/kylin/pull/1796
--
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: issues-unsub
zhangayqian merged pull request #1795:
URL: https://github.com/apache/kylin/pull/1795
--
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: issues-unsub
zhangayqian opened a new pull request #1795:
URL: https://github.com/apache/kylin/pull/1795
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus merged pull request #1794:
URL: https://github.com/apache/kylin/pull/1794
--
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: issues-unsubsc
nichunen opened a new pull request #1794:
URL: https://github.com/apache/kylin/pull/1794
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be su
hit-lacus merged pull request #1792:
URL: https://github.com/apache/kylin/pull/1792
--
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: issues-unsubsc
hit-lacus commented on a change in pull request #1793:
URL: https://github.com/apache/kylin/pull/1793#discussion_r782672366
##
File path: website/_posts/blog/2022-01-12-The-Future-Of-Kylin.cn.md
##
@@ -0,0 +1,47 @@
+---
+layout: post-blog
+title: 下一代 Kylin:更强大和易用的 OLAP
+date: 2
zhangayqian opened a new pull request #1793:
URL: https://github.com/apache/kylin/pull/1793
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus commented on pull request #1791:
URL: https://github.com/apache/kylin/pull/1791#issuecomment-1010575381
Thanks @sleep1661 !
--
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
nichunen opened a new pull request #1792:
URL: https://github.com/apache/kylin/pull/1792
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be su
sleep1661 commented on pull request #1791:
URL: https://github.com/apache/kylin/pull/1791#issuecomment-1009545902
@hit-lacus @zhangayqian @zhengshengjun could you help take a look when you
have time? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the m
sleep1661 commented on pull request #1790:
URL: https://github.com/apache/kylin/pull/1790#issuecomment-1009545765
@hit-lacus @zhangayqian @zhengshengjun could you help take a look when you
have time? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the m
sleep1661 opened a new pull request #1791:
URL: https://github.com/apache/kylin/pull/1791
## Proposed changes
Support boolean expression (true or false) on where clause. For example
`select SELLER_ID,sum(PRICE) from KYLIN_SALES where true GROUP BY SELLER_ID `
## Github Branch
codecov-commenter commented on pull request #1790:
URL: https://github.com/apache/kylin/pull/1790#issuecomment-1008730029
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1790?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
sleep1661 opened a new pull request #1790:
URL: https://github.com/apache/kylin/pull/1790
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be s
zhangayqian merged pull request #1789:
URL: https://github.com/apache/kylin/pull/1789
--
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: issues-unsub
zhangayqian merged pull request #1789:
URL: https://github.com/apache/kylin/pull/1789
--
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: issues-unsub
zhangayqian opened a new pull request #1789:
URL: https://github.com/apache/kylin/pull/1789
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus merged pull request #1787:
URL: https://github.com/apache/kylin/pull/1787
--
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: issues-unsubsc
smartbooks closed pull request #1788:
URL: https://github.com/apache/kylin/pull/1788
--
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: issues-unsubs
smartbooks opened a new pull request #1788:
URL: https://github.com/apache/kylin/pull/1788
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian opened a new pull request #1787:
URL: https://github.com/apache/kylin/pull/1787
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus merged pull request #1786:
URL: https://github.com/apache/kylin/pull/1786
--
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: issues-unsubsc
hit-lacus merged pull request #1784:
URL: https://github.com/apache/kylin/pull/1784
--
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: issues-unsubsc
hit-lacus merged pull request #1785:
URL: https://github.com/apache/kylin/pull/1785
--
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: issues-unsubsc
zhangayqian opened a new pull request #1786:
URL: https://github.com/apache/kylin/pull/1786
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian opened a new pull request #1785:
URL: https://github.com/apache/kylin/pull/1785
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
zhangayqian opened a new pull request #1784:
URL: https://github.com/apache/kylin/pull/1784
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus merged pull request #1783:
URL: https://github.com/apache/kylin/pull/1783
--
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: issues-unsubsc
zhangayqian opened a new pull request #1783:
URL: https://github.com/apache/kylin/pull/1783
## Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request. If it fixes a bug or
resolves a feature request, be
hit-lacus commented on a change in pull request #1779:
URL: https://github.com/apache/kylin/pull/1779#discussion_r776196021
##
File path:
source-hive/src/main/java/org/apache/kylin/source/hive/MRHiveDictUtil.java
##
@@ -114,33 +122,39 @@ public static String generateDropTableS
zhangayqian edited a comment on pull request #1778:
URL: https://github.com/apache/kylin/pull/1778#issuecomment-1002430840
@fengpod
The following three problems were found during the test:
1. Step `Build Hive Global Dict With Spark` has actually made an error, but
it still shows
zhangayqian commented on pull request #1778:
URL: https://github.com/apache/kylin/pull/1778#issuecomment-1002430840
@fengpod
The following three problems were found during the test:
1. Step `Build Hive Global Dict With Spark` has actually made an error, but
it still shows success
zhangayqian commented on a change in pull request #1778:
URL: https://github.com/apache/kylin/pull/1778#discussion_r776139094
##
File path:
source-hive/src/main/java/org/apache/kylin/source/hive/CreateSparkHiveDictStep.java
##
@@ -0,0 +1,294 @@
+/*
+ * Licensed to the Apache S
codecov-commenter edited a comment on pull request #1780:
URL: https://github.com/apache/kylin/pull/1780#issuecomment-989795370
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1780?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #1780:
URL: https://github.com/apache/kylin/pull/1780#issuecomment-989795370
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1780?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
fengpod commented on pull request #1778:
URL: https://github.com/apache/kylin/pull/1778#issuecomment-1001896249
@hit-lacus i have fixed it,and resolved conflicts
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
hit-lacus merged pull request #1782:
URL: https://github.com/apache/kylin/pull/1782
--
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: issues-unsubsc
901 - 1000 of 1558 matches
Mail list logo