Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21675#discussion_r199316192
--- Diff: python/pyspark/ml/util.py ---
@@ -63,7 +63,7 @@ def _randomUID(cls):
Generate a unique unicode id for the object. The default
implementation
concatenates the class name, "_", and 12 random hex chars.
"""
- return unicode(cls.__name__ + "_" + uuid.uuid4().hex[12:])
+ return unicode(cls.__name__ + "_" + uuid.uuid4().hex[:12])
--- End diff --
`uuid.uuid4().hex[-12:]`? In Scala side, it is
`UUID.randomUUID().toString.takeRight(12)`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
