Re: Apache Kylin FAQ updated

2018-10-02 Thread Li Yang 
Nice~~

On Wed, Sep 19, 2018 at 10:03 AM ShaoFeng Shi 
wrote:

> Hello,
>
> The FAQ page got updated together with v2.5.0 release; Please refresh to
> see the changes:
>
> https://kylin.apache.org/docs/gettingstarted/faq.html
>
> If you have a question to be added to this page, just let me know.
> --
> Best regards,
>
> Shaofeng Shi 史少锋
>

Re: [DISCUSS] Not sending Github PR notifications to dev@kylin

2018-10-02 Thread Li Yang 
+1 make sense

On Tue, Oct 2, 2018 at 11:38 AM ShaoFeng Shi  wrote:

> Hello, Kylin dev subscribers,
>
> Recently I received several complaints saying that there are many emails
> sent to the "dev@kylin.apache.org" from the github.com pull request since
> we enabled the Gitbox service for Kylin.
>
> Today most patches and code reviews are performed on GitHub. Each pull
> request action (even add a comment) will emit an email to dev@kylin,
> instead of the individual contributor or reviewer; This generates many
> spams and causes the emails from people are left in the basket.
>
> Now I plan to change the Gitbox email notifications rule: removing
> dev@kylin,
> use author and reviewer instead, as follows:
>
>
> *For Github issues, please notify iss...@kylin.apache.org
>  ;For Github PR, please notify the author,
> reviewer and iss...@kylin.apache.org *
>
> The related JIRA to Apache Infra is
> https://issues.apache.org/jira/browse/INFRA-17073
>
> Please +1 if you agree with the new rule, or -1 if you want to keep as
> today. If no objection, we will move on with the new rule.
>
> --
> Best regards,
>
> Shaofeng Shi 史少锋
>

[jira] [Created] (KYLIN-3609) NPE from QueryMetricsFacade#updateMetricsToReservoir

2018-10-02 Thread Ted Yu (JIRA) 
Ted Yu created KYLIN-3609:
-

 Summary: NPE from QueryMetricsFacade#updateMetricsToReservoir
 Key: KYLIN-3609
 URL: https://issues.apache.org/jira/browse/KYLIN-3609
 Project: Kylin
  Issue Type: Bug
Reporter: Ted Yu


When running test suite, I saw the following in test output:
{code}
2018-10-02 20:59:20,415 WARN  [Query c8f42f2e-8c77-2bfc-97ab-053fbeb7c86e-1] 
service.QueryService:423 : Write metric error.
java.lang.NullPointerException
  at 
org.apache.kylin.rest.metrics.QueryMetricsFacade.updateMetricsToReservoir(QueryMetricsFacade.java:148)
  at 
org.apache.kylin.rest.metrics.QueryMetricsFacade.updateMetrics(QueryMetricsFacade.java:74)
  at 
org.apache.kylin.rest.service.QueryService.recordMetric(QueryService.java:505)
  at 
org.apache.kylin.rest.service.QueryService.doQueryWithCache(QueryService.java:421)
  at 
org.apache.kylin.rest.service.QueryService.doQueryWithCache(QueryService.java:353)
  at 
org.apache.kylin.rest.controller.QueryController.query(QueryController.java:87)
  at 
org.apache.kylin.rest.controller.QueryControllerTest.testQueryException(QueryControllerTest.java:63)
{code}
It seems sqlResponse.getResults() returned null.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Created] (KYLIN-3608) Move dependency versions to top level pom properties

2018-10-02 Thread Ted Yu (JIRA) 
Ted Yu created KYLIN-3608:
-

 Summary: Move dependency versions to top level pom properties
 Key: KYLIN-3608
 URL: https://issues.apache.org/jira/browse/KYLIN-3608
 Project: Kylin
  Issue Type: Task
Reporter: Ted Yu


There are some non-top level pom.xml files where dependency version is 
referenced directly.

core-common/pom.xml is an example.

We should move all dependency versions to top level pom properties



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)