Zoltan Borok-Nagy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24931 )

Change subject: IMPALA-15214: [DOCS] Update CPU requirements
......................................................................


Patch Set 2:

(4 comments)

Thanks for the comments!

http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_incompatible_changes.xml
File docs/topics/impala_incompatible_changes.xml:

http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_incompatible_changes.xml@82
PS1, Line 82: On x86-64 CPUs, the Impala daemons do not start
            :             without the AVX2 instruction set. Earlier
> On x86-64 CPUs, the Impala daemons do not start without the AVX2 instructio
Done


http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_prereqs.xml
File docs/topics/impala_prereqs.xml:

http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_prereqs.xml@a291
PS1, Line 291:
             :
             :
             :
             :
             :
             :
> Do we have any current benchmark that suggest similar CPU families? I think
I'm not aware of such benchmarks, and doing such measurements would deserve a 
separete ticket.

I think this recommendation was more important in the early days when Impala 
used features of then new CPUs, so they had an edge over other/older CPUs. But 
nowadays AVX2 and CRC32 are basically everywhere. Also providing such 
recommendations can be counter-productive as they might give the impression 
that you need those specific CPUs to get a really performant Impala.


http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_prereqs.xml@266
PS1, Line 266:  a
> nit: comma not needed
Done


http://gerrit.cloudera.org:8080/#/c/24931/1/docs/topics/impala_prereqs.xml@281
PS1, Line 281: with the <codeph>
> nit: not needed
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I413ac12719cbbcdd4cbdf99e77f24ab10d41651d
Gerrit-Change-Number: 24931
Gerrit-PatchSet: 2
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Thu, 24 Sep 2026 09:28:20 +0000
Gerrit-HasComments: Yes

Reply via email to