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.

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

HPCC-16816 Copy original password to cached secuser #9456

Merged

Conversation

wangkx
Copy link
Member

@wangkx wangkx commented Jan 4, 2017

Signed-off-by: wangkx kevin.wang@lexisnexis.com

Signed-off-by: wangkx <kevin.wang@lexisnexis.com>
@hpcc-jirabot
Copy link

@wangkx
Copy link
Member Author

wangkx commented Jan 4, 2017

@afishbeck please review.

@HPCCSmoketest
Copy link
Contributor

Automated Smoketest
Sha: 40a7b92
Build: success
ECL Watch: Rebuilding Site

errors warnings build time
0 65 91.027 seconds

Install hpccsystems-platform-community_6.2.1-closedown0.el7.x86_64.rpm
HPCC Start: OK

Unittest result:

total passed failed timeout
88 88 0 0

HPCC Stop: OK
HPCC Uninstall: OK

Copy link
Member

@afishbeck afishbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. The new variable name is helpful too.

@wangkx
Copy link
Member Author

wangkx commented Jan 4, 2017

@richardkchapman this PR is ready to be merged.

@RussWhitehead
Copy link
Member

@wangkx @afishbeck Looks good, and I tested to confirm

@richardkchapman richardkchapman merged commit 7756036 into hpcc-systems:candidate-6.2.2 Jan 5, 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
6 participants