Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[data] Basic structured logging #47210

Merged
merged 10 commits into from
Oct 11, 2024
Merged

[data] Basic structured logging #47210

merged 10 commits into from
Oct 11, 2024

Commits on Oct 11, 2024

  1. basic structured logging 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    b5271b7 View commit details
    Browse the repository at this point in the history

  2. respond to feedback 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    dd44baa View commit details
    Browse the repository at this point in the history

  3. Update python/ray/data/_internal/logging.py 

    Co-authored-by: Scott Lee <scottjlee@users.noreply.github.com>
Signed-off-by: Matthew Owen <omatthew98@berkeley.edu>
    @omatthew98 @scottjlee
    omatthew98 and scottjlee committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    517eda8 View commit details
    Browse the repository at this point in the history

  4. Update python/ray/data/_internal/logging.py 

    Co-authored-by: Hao Chen <chenh1024@gmail.com>
Signed-off-by: Matthew Owen <omatthew98@berkeley.edu>
    @omatthew98 @raulchen
    omatthew98 and raulchen committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    9450689 View commit details
    Browse the repository at this point in the history

  5. JSON mode log text to console 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    8511d37 View commit details
    Browse the repository at this point in the history

  6. update code to use single config, tweak logic to accommodate this 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    9608e3b View commit details
    Browse the repository at this point in the history

  7. adding in test and dynamic env var loading 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    e986017 View commit details
    Browse the repository at this point in the history

  8. swap class for () to make corecontextfilter work 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    1216470 View commit details
    Browse the repository at this point in the history

  9. pr feedback 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    a2e4377 View commit details
    Browse the repository at this point in the history

  10. update env var name 

    Signed-off-by: Matthew Owen <mowen@anyscale.com>
    @omatthew98
    omatthew98 committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    b4e0e6a View commit details
    Browse the repository at this point in the history