https://pinot.apache.org/ logo
Join Slack
Powered by
# general
  • j

    James Shao

    05/01/2019, 11:15 PM
    yeah I think we (Uber) use log4j instead of log4j2
  • j

    James Shao

    05/01/2019, 11:21 PM
    maybe there is some backward compatible version? usually, I see the disk write being the bottleneck and removing that helps a lot as well (use async logger)
  • s

    Seunghyun

    05/01/2019, 11:41 PM
    if i check-in this, it will pick up the default setting until you write a new log4j2 config and pass it as a jvm option
  • k

    Kishore G

    05/02/2019, 2:11 AM
    What is the default setting?
  • s

    Seunghyun

    05/02/2019, 5:22 AM
    Copy code
    Log4j will provide a default configuration if it cannot locate a configuration file. The default configuration, provided in the DefaultConfiguration class, will set up:
    
    A ConsoleAppender attached to the root logger.
    A PatternLayout set to the pattern "%d{HH:mm:ss.SSS} [%t] %-5level %logger{36} - %msg%n" attached to the ConsoleAppender
    Note that by default Log4j assigns the root logger to Level.ERROR.
    Copy code
    The output of MyApp would be similar to:
    
    17:13:01.540 [main] ERROR com.foo.Bar - Did it again!
    17:13:01.540 [main] ERROR MyApp - Didn't do it.
    https://logging.apache.org/log4j/2.x/manual/configuration.html
  • s

    Seunghyun

    05/02/2019, 5:22 AM
    it will output the log to console, set the level to
    error
    , and the format is like the above example
  • j

    James Shao

    05/02/2019, 4:23 PM
    just curious have we considered using the log4j 1->2 bridge/compatible library instead of direct migration, such as this: https://logging.apache.org/log4j/log4j-2.2/log4j-1.2-api/index.html
  • m

    Mayank

    05/02/2019, 4:23 PM
    @snlee ^^
  • s

    Subbu Subramaniam

    05/02/2019, 4:37 PM
    @User ^^
  • m

    Mayank

    05/02/2019, 4:37 PM
    Thanks @User for correcting
  • s

    Seunghyun

    05/02/2019, 5:23 PM
    If you check the pr, we do import log4j 1->2 bridge. But, this is for routing log4j 1.x apis to log4j2. This is more for making the code backward compatible, not the config (if you coded your application using log4j 1.x api, importing this will redirect log4j1.x api to log4j2). In our case, we use slf4j for logging api so it doesn’t apply but I pulled that bridge because some of our dependencies are directly using log4j 1.x api.
  • s

    Seunghyun

    05/02/2019, 5:23 PM
    @User ^^
  • s

    Seunghyun

    05/02/2019, 5:26 PM
    https://logging.apache.org/log4j/log4j-2.2/manual/migration.html
  • s

    Seunghyun

    05/02/2019, 10:01 PM
    If this code goes to the official apache release to 0.2.0, we will include this information to the release note.
    👍 1
  • a

    Andre Hsu

    05/03/2019, 7:55 PM
    Hello everyone! I was wondering if anyone ran into controller.out taking up disk space issue. Pinot controller logs don't seem to clear or rotate by default. Any suggestions on how to set that? thank you!
  • a

    anshul.bansal

    05/06/2019, 8:05 PM
    Hello Everyone ! Has anyone added their own jdbc datasource to third eye
  • a

    anshul.bansal

    05/06/2019, 8:05 PM
    ?
  • k

    Kishore G

    05/06/2019, 8:48 PM
    Will respond in few hours
  • a

    anshul.bansal

    05/07/2019, 1:47 AM
    @User @User, Thanks for the revert. few insights: 1. Currently app/#/self-serve/import-sql-metric is not reachable directly from create alert page. 2. mysqlDBNameToDataSourceMap in SqlResponseCacheLoader is not populated with datasource and is null.
  • a

    anshul.bansal

    05/07/2019, 1:52 AM
    I have added redshift jdbc connection like this
    data-source-config_yml.yaml
  • k

    Kishore G

    05/07/2019, 1:53 AM
    there is another slack channel for thirdeye
  • a

    anshul.bansal

    05/07/2019, 1:55 AM
    👍
  • s

    Seunghyun

    05/07/2019, 10:27 PM
    <!here> I merged log4j2 migration commit. If you are deploying after this change, you will need to rewrite log4j configuration file to log4j2 format. We will update note on this for the next apache release (0.2.0). https://github.com/apache/incubator-pinot/pull/4139
    👍 1
  • c

    Chinmay Soman

    05/08/2019, 9:30 PM
    Hey folks, is there any thinking around automatically discarding "null" (default) values when querying Pinot. I think the current guidance we give to customers is that filter out explicitly in your query which is a bit inconvenient.
  • k

    Kishore G

    05/08/2019, 9:31 PM
    you mean support for NULL ?
  • c

    Chinmay Soman

    05/08/2019, 9:43 PM
    yes
  • i

    Ivan Korhner

    05/10/2019, 12:21 PM
    Hey guys, I am trying to import a metric to ThirdEye that is calculated by dividing two numbers (a percentage). I can't create a pinot column that has that value calculated as percentage is not additive and breakdown by dimensions will not be possible. Is there a way to define a metric in ThirdEye that is calculated by combining other metrics on the fly?
  • i

    Ivan Korhner

    05/10/2019, 12:23 PM
    A concrete example: day 1 retention (number of users who logged in today and registered yesterday / total number of users registered yesterday). I would like to create alerts on that metric and also be able to break down by all dimensions in root cause analysis
  • k

    Kishore G

    05/10/2019, 4:04 PM
    Yes. There is concept of derived metric and you can setup the expression to compute it on the admin UI/rest api.
  • i

    Ivan Korhner

    05/10/2019, 6:03 PM
    Perfect, thanks! Are derived metrics documented anywhere? Most of it is clear, but for example no idea what is Inverse metric supposed to be?
1...777879...160Latest