Wenzhe Zhou has uploaded this change for review. ( 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 --- 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(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/64/16564/1 -- 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: newchange Gerrit-Change-Id: I22b1e91cf9d6c89a3c62749ae0fd88528ae69885 Gerrit-Change-Number: 16564 Gerrit-PatchSet: 1 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
