Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16564 )
Change subject: IMPALA-10207: Avoid MD5 hash for lineage graph ...................................................................... IMPALA-10207: Avoid MD5 hash for lineage graph To support FIPS, we have to avoid MD5 since it's not allowed algorithms for FIPS. But we use MD5 hash for lineage graph. This patch replace MD5 with non-cryptographic hash function murmur3_128 which generates hash value with same length as MD5. External dependency on the hash value: Went through Apache Atlas source code. ImpalaQuery.getHash() function (https://github.com/apache/atlas/blob/master/addons/impala-bridge/src/ main/java/org/apache/atlas/impala/model/ImpalaQuery.java#L60) is not called anywhere. Don't see any dependency on the hash value in Atlas. Testing: - Passed test_lineage.py. - Passed core tests. Change-Id: I22b1e91cf9d6c89a3c62749ae0fd88528ae69885 Reviewed-on: http://gerrit.cloudera.org:8080/16564 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M common/thrift/LineageGraph.thrift M fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java M testdata/workloads/functional-query/queries/QueryTest/lineage.test 3 files changed, 45 insertions(+), 45 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16564 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I22b1e91cf9d6c89a3c62749ae0fd88528ae69885 Gerrit-Change-Number: 16564 Gerrit-PatchSet: 2 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
