[GitHub] [kylin] hit-lacus commented on pull request #1813: [KYLIN-4382] Unable to use DATE type in prepared statements in kylin3

2022-02-08 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1813: [KYLIN-4382] Unable to use DATE type in prepared statements in kylin3

2022-02-08 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1805: KYLIN-5157 Attach username and password to README.md

2022-02-08 Thread GitBox
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

[GitHub] [kylin] hit-lacus commented on pull request #1805: KYLIN-5157 Attach username and password to README.md

2022-02-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1813: [KYLIN-4382] Unable to use DATE type in prepared statements in kylin3

2022-02-08 Thread GitBox
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

[GitHub] [kylin] pjfanning opened a new pull request #1815: [KYLIN-5159] jackson.databind.version=2.6.7.5

2022-02-08 Thread GitBox
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

[GitHub] [kylin] codecov-commenter commented on pull request #1814: [KYLIN-5159] upgrade some common libs due to CVEs

2022-02-08 Thread GitBox
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

[GitHub] [kylin] pjfanning opened a new pull request #1814: [KYLIN-5159] upgrade some common libs due to CVEs

2022-02-08 Thread GitBox
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

[GitHub] [kylin] juntaozhang commented on pull request #1813: [KYLIN-4382] Unable to use DATE type in prepared statements in kylin3

2022-02-08 Thread GitBox
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

[GitHub] [kylin] juntaozhang closed pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-08 Thread GitBox
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

[GitHub] [kylin] juntaozhang opened a new pull request #1813: [KYLIN-4382] Unable to use DATE type in prepared statements in kylin3

2022-02-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-08 Thread GitBox
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

[GitHub] [kylin] juntaozhang commented on pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on a change in pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-08 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1812: # update README.md

2022-02-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1811: fix

2022-02-08 Thread GitBox
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

[GitHub] [kylin] hit-lacus commented on a change in pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-07 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1810: adapt initialized config for default aws account limits

2022-02-07 Thread GitBox
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

[GitHub] [kylin] lgtm-com[bot] commented on pull request #1810: adapt initialized config for default aws account limits

2022-02-07 Thread GitBox
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://

[GitHub] [kylin] Mukvin opened a new pull request #1810: adapt initialized config for default aws account limits

2022-02-07 Thread GitBox
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

[GitHub] [kylin] codecov-commenter edited a comment on pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-06 Thread GitBox
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=

[GitHub] [kylin] codecov-commenter commented on pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-06 Thread GitBox
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

[GitHub] [kylin] juntaozhang opened a new pull request #1809: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-06 Thread GitBox
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

[GitHub] [kylin] juntaozhang closed pull request #1808: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-06 Thread GitBox
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

[GitHub] [kylin] juntaozhang opened a new pull request #1808: [KYLIN-4382] Unable to use DATE type in prepared statements

2022-02-06 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1807: # support multiple region for tool

2022-02-02 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1807: # support multiple region for tool

2022-02-02 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1806: # update README.md

2022-01-28 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1806: # update README.md

2022-01-27 Thread GitBox
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

[GitHub] [kylin] codecov-commenter edited a comment on pull request #1805: KYLIN-5157 Attach username and password to README.md

2022-01-26 Thread GitBox
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=

[GitHub] [kylin] codecov-commenter commented on pull request #1805: KYLIN-5157 Attach username and password to README.md

2022-01-26 Thread GitBox
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

[GitHub] [kylin] zhouyao1994 opened a new pull request #1805: KYLIN-5157 Attach username and password to README.md

2022-01-26 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1803: Fix readme 1

2022-01-25 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1804: # update README.md && fix some typo

2022-01-25 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1802: # update README.md

2022-01-25 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1804: # update README.md && fix some typo

2022-01-25 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1804: # update README.md && fix some typo

2022-01-25 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1803: Fix readme 1

2022-01-24 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1803: Fix readme 1

2022-01-24 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1802: # update README.md

2022-01-24 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1802: # update README.md

2022-01-24 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1801: # init && deploy kylin4 on aws

2022-01-23 Thread GitBox
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

[GitHub] [kylin] Mukvin opened a new pull request #1801: # init && deploy kylin4 on aws

2022-01-23 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1798: Added some tips on spark

2022-01-19 Thread GitBox
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

[GitHub] [kylin] sleep1661 commented on pull request #1800: [KYLIN-5155] Set volatile range value be 7 by default

2022-01-19 Thread GitBox
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

[GitHub] [kylin] sleep1661 opened a new pull request #1800: [KYLIN-5155] Set volatile range value be 7 by default

2022-01-19 Thread GitBox
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

[GitHub] [kylin] zhangayqian merged pull request #1799: Replace kyligence logo

2022-01-19 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1799: Replace kyligence logo

2022-01-19 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1798: Added some tips on spark

2022-01-19 Thread GitBox
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

[GitHub] [kylin] hit-lacus closed pull request #1793: Add new blog: The future of Kylin

2022-01-19 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1797: Add new blog: The future of Kylin

2022-01-19 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1767: [KYLIN-5121] Make JobMetricsUtils.collectMetrics be working again

2022-01-19 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1770: [KYLIN-5125] `MetadataCleanupJob.isJobComplete` should use `NSparkCubingJob` because kylin4 use spark engine to build cube

2022-01-19 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1791: [KYLIN-5152] support boolean expression on where clause

2022-01-19 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1797: Add new blog: The future of Kylin

2022-01-19 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1791: [KYLIN-5152] support boolean expression on where clause

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1790: [KYLIN-5090] isExactlyMatchSet should keep same size with cuboidIds

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1780: KYLIN-5141 hive global dict replace with spark

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on a change in pull request #1780: KYLIN-5141 hive global dict replace with spark

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on a change in pull request #1780: KYLIN-5141 hive global dict replace with spark

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1796: Fix format

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian merged pull request #1796: Fix format

2022-01-13 Thread GitBox
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

[GitHub] [kylin] zhangayqian merged pull request #1795: Fix format

2022-01-12 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1795: Fix format

2022-01-12 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1794: Remove code styles

2022-01-12 Thread GitBox
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

[GitHub] [kylin] nichunen opened a new pull request #1794: Remove code styles

2022-01-12 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1792: Add doc about how to set up front end registry

2022-01-12 Thread GitBox
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

[GitHub] [kylin] hit-lacus commented on a change in pull request #1793: Add new blog: The future of Kylin

2022-01-11 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1793: Add new blog: The future of Kylin

2022-01-11 Thread GitBox
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

[GitHub] [kylin] hit-lacus commented on pull request #1791: [KYLIN-5152] support boolean expression on where clause

2022-01-11 Thread GitBox
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

[GitHub] [kylin] nichunen opened a new pull request #1792: Add doc about how to set up front end registry

2022-01-11 Thread GitBox
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

[GitHub] [kylin] sleep1661 commented on pull request #1791: [KYLIN-5152] support boolean expression on where clause

2022-01-10 Thread GitBox
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

[GitHub] [kylin] sleep1661 commented on pull request #1790: [KYLIN-5090] isExactlyMatchSet should keep same size with cuboidIds

2022-01-10 Thread GitBox
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

[GitHub] [kylin] sleep1661 opened a new pull request #1791: [KYLIN-5152] support boolean expression on where clause

2022-01-10 Thread GitBox
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

[GitHub] [kylin] codecov-commenter commented on pull request #1790: [KYLIN-5090] isExactlyMatchSet should keep same size with cuboidIds

2022-01-10 Thread GitBox
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

[GitHub] [kylin] sleep1661 opened a new pull request #1790: [KYLIN-5090] isExactlyMatchSet should keep same size with cuboidIds

2022-01-10 Thread GitBox
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

[GitHub] [kylin] zhangayqian merged pull request #1789: Fix some link

2022-01-08 Thread GitBox
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

[GitHub] [kylin] zhangayqian merged pull request #1789: Fix some link

2022-01-07 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1789: Fix some link

2022-01-07 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1787: Update security page

2022-01-06 Thread GitBox
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

[GitHub] [kylin] smartbooks closed pull request #1788: add aliyun emr support

2022-01-06 Thread GitBox
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

[GitHub] [kylin] smartbooks opened a new pull request #1788: V4.0.1 release

2022-01-06 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1787: Update security page

2022-01-05 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1786: Release 4.0.1 and 3.1.3

2022-01-04 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1784: V3.1.3 release

2022-01-04 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1785: V4.0.1 release

2022-01-04 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1786: Release 4.0.1 and 3.1.3

2022-01-04 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1785: V4.0.1 release

2022-01-04 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1784: V3.1.3 release

2022-01-04 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1783: KYLIN-5097 Fix bug

2021-12-29 Thread GitBox
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

[GitHub] [kylin] zhangayqian opened a new pull request #1783: KYLIN-5097 Fix bug

2021-12-29 Thread GitBox
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

[GitHub] [kylin] hit-lacus commented on a change in pull request #1779: KYLIN-5097 configurable hive dict table format

2021-12-29 Thread GitBox
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

[GitHub] [kylin] zhangayqian edited a comment on pull request #1778: KYLIN-5097 spark hive global dict

2021-12-28 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on pull request #1778: KYLIN-5097 spark hive global dict

2021-12-28 Thread GitBox
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

[GitHub] [kylin] zhangayqian commented on a change in pull request #1778: KYLIN-5097 spark hive global dict

2021-12-28 Thread GitBox
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

[GitHub] [kylin] codecov-commenter edited a comment on pull request #1780: KYLIN-5141 hive global dict replace with spark

2021-12-28 Thread GitBox
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

[GitHub] [kylin] codecov-commenter edited a comment on pull request #1780: KYLIN-5141 hive global dict replace with spark

2021-12-28 Thread GitBox
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

[GitHub] [kylin] fengpod commented on pull request #1778: KYLIN-5097 spark hive global dict

2021-12-27 Thread GitBox
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

[GitHub] [kylin] hit-lacus merged pull request #1782: test fix

2021-12-26 Thread GitBox
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

<    5   6   7   8   9   10   11   12   13   14   >