Uploaded image for project: 'Zeppelin'
  1. Zeppelin
  2. ZEPPELIN-3042

Outdated git library doesn't support post-commit hooks

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 0.7.3, 0.8.0, 0.7.4
    • 0.8.0
    • zeppelin-zengine
    • Amazon EMR

    • Patch

    Description

      We run Zeppelin (0.7.2, on Amazon's EMR) and we tried adding git hooks, primarily to push the local notebook git repo to our remote. We noticed that post commit hooks don't get triggered. When I dug around, I noticed that pre-commits worked just fine.

      Turns out that when git functionality was first implemented in late 2015, `jgit` (library used to provide all git-related functionality) was frozen at version 4.1.1, which you can see in the current `pom.xml`. However, `jgit` has gained post-commit functionality over a year ago, so Zeppelin could support it as well, if we updated the version of `jgit` that gets used. The current version is v4.9.0.201710071750-r (7 Oct 2017).

      Attachments

        Issue Links

          Activity

            People

              ondrej Ondrej Kokes
              ondrej Ondrej Kokes
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: