Github user pu239ppy commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/307#discussion_r201482215
  
    --- Diff: phoenix-queryserver/src/it/bin/test_phoenixdb.sh ---
    @@ -0,0 +1,59 @@
    +#/usr/bin/env bash
    +
    +set -u
    +set -x
    +set -e
    +
    +function cleanup {
    +    set +e
    +    set +u
    +    kdestroy
    --- End diff --
    
    Not to be overly pedantic, but you would want to still pass krb5ccname and 
just call kdestroy to make sure proper cleanup is done


---

Reply via email to