Skip to content

#2506 use KerberosToken if accumulo client is configured to use it - #2510

Merged
echeipesh merged 1 commit into
locationtech:1.2from
VitoTAP:1.2
Dec 11, 2017
Merged

echeipesh merged 1 commit into
locationtech:1.2from
VitoTAP:1.2

Conversation

@jdries

@jdries jdries commented Dec 11, 2017

Copy link
Copy Markdown
Contributor

Please consider this pull request for inclusion in version 1.2.0, it allows people with a kerberized accumulo to use geopyspark.

@pomadchin

Copy link
Copy Markdown
Member

Hi! Thanks for you contribution a lot. We'd be happy to try to include your PR even into 1.2.0. Before we dived into the review process, can you sign our CLA?

Instructions to sign up Eclipse CLA (if you haven't already signed it): CONTRIBUTING.rst

After that to sign the current commit (a quick tip):

git commit --amend -s && git push origin -f master

@jdries

jdries commented Dec 11, 2017

Copy link
Copy Markdown
Contributor Author

Hi, I signed the CLA, and it seems that the ip-validation check now passes!

@pomadchin

Copy link
Copy Markdown
Member

@jdries thank you!

@echeipesh
echeipesh merged commit 743256b into locationtech:1.2 Dec 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants