Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Feb 20, 2018

The secret is used as a string in many parts of the code, so it has
to be turned into a hex string to avoid issues such as the random
byte sequence not containing a valid UTF8 sequence.

…tials.

The secret is used as a string in many parts of the code, so it has
to be turned into a hex string to avoid issues such as the random
byte sequence not containing a valid UTF8 sequence.
@vanzin
Copy link
Contributor Author

vanzin commented Feb 20, 2018

@jerryshao @tgravescs

@tgravescs
Copy link
Contributor

+1 pending the QA build.

@SparkQA
Copy link

SparkQA commented Feb 21, 2018

Test build #87566 has finished for PR 20643 at commit 77d009d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

asfgit pushed a commit that referenced this pull request Feb 21, 2018
…tials.

The secret is used as a string in many parts of the code, so it has
to be turned into a hex string to avoid issues such as the random
byte sequence not containing a valid UTF8 sequence.

Author: Marcelo Vanzin <[email protected]>

Closes #20643 from vanzin/SPARK-23468.

(cherry picked from commit 6d398c0)
Signed-off-by: Marcelo Vanzin <[email protected]>
@asfgit asfgit closed this in 6d398c0 Feb 21, 2018
@vanzin vanzin deleted the SPARK-23468 branch March 5, 2018 23:54
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
…tials.

The secret is used as a string in many parts of the code, so it has
to be turned into a hex string to avoid issues such as the random
byte sequence not containing a valid UTF8 sequence.

Author: Marcelo Vanzin <[email protected]>

Closes apache#20643 from vanzin/SPARK-23468.

(cherry picked from commit 6d398c0)
Signed-off-by: Marcelo Vanzin <[email protected]>
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