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

[GOBBLIN-2168] Add TimeBasedSnapshotCleanupPolicyTest and Fix a bug with props.containsKey() instead of props.contains() in Trash class #4070

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

[GOBBLIN-2168] Add TimeBasedSnapshotCleanupPolicyTest and Fix a bug with props.containsKey() instead of props.contains() in Trash class #4070

wants to merge 9 commits into from

Commits on Oct 30, 2024

  1. Compare with the current time in UTC

    @linweihs
    linweihs committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    ed01119 View commit details
    Browse the repository at this point in the history

  2. clean up code

    @linweihs
    linweihs committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    6bfb281 View commit details
    Browse the repository at this point in the history

  3. address comments

    @linweihs
    linweihs committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    e864d80 View commit details
    Browse the repository at this point in the history

  4. update unit tests

    @linweihs
    linweihs committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    0ab6daa View commit details
    Browse the repository at this point in the history

Commits on Nov 9, 2024

  1. address comments

    @linweihs
    linweihs committed Nov 9, 2024
    Configuration menu
    Copy the full SHA
    3ead802 View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2024

  1. address comment

    @linweihs
    linweihs committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    c5c4892 View commit details
    Browse the repository at this point in the history

  2. address comments

    @linweihs
    linweihs committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    401fb30 View commit details
    Browse the repository at this point in the history

Commits on Nov 20, 2024

  1. Fix bug where azkaban prop.contains(object) is checking value not key… 

    … as intended
    @linweihs
    linweihs committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    f749ab9 View commit details
    Browse the repository at this point in the history

  2. Mock the FileStatus object to return the desired path

    @linweihs
    linweihs committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    5b6fa8f View commit details
    Browse the repository at this point in the history