Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15531 )

Change subject: IMPALA-9544 Replace Intel's SSE instructions with ARM's NEON 
instructions
......................................................................


Patch Set 44:

(1 comment)

I looked at the startup failure and the problem was pretty clear, so I will 
rerun with that part of the change removed.

http://gerrit.cloudera.org:8080/#/c/15531/44/be/src/codegen/gen_ir_descriptions.py
File be/src/codegen/gen_ir_descriptions.py:

http://gerrit.cloudera.org:8080/#/c/15531/44/be/src/codegen/gen_ir_descriptions.py@245
PS44, Line 245: if platform.processor() != "aarch64":
I missed that this function was removed in 
http://gerrit.cloudera.org:8080/15683. So I don't think we need to modify this 
file. That's causing the startup failure cause the file no longer exists



--
To view, visit http://gerrit.cloudera.org:8080/15531
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id7dfe17125b2910ece54e7dd18b4e4b25d7de8b9
Gerrit-Change-Number: 15531
Gerrit-PatchSet: 44
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Fri, 14 Aug 2020 19:37:23 +0000
Gerrit-HasComments: Yes

Reply via email to