r-funke

Results 6 comments of r-funke

@jglick thank you for your input. I've moved AdditionalCredentials back in place. Regarding the non-deterministic revision, it was an easy solution (or workaround) for Jenkins not detecting changes in externals....

Would it help to remove the changes to SCMRevisionImpl and isDeterministic() from this PR to get it accepted? Then this PR would only cover AdditionalCredentials for SubversionSCMSource, i.e. for multibranch...

Okay, this PR is now only about additional credentials. Is it good enough to be accepted?

I did check with Pulsar 2.7.2 now. It doesn't work either, but the behavior is a little bit different. The sql-worker doesn't crash, but there is `NULL` values returned by...

I think I somehow messed up the check on Pulsar 2.7.2 in my previous reply. I said that it returned null values instead of the real data in sql queries....