Using select * could cause SSIS to crash if a column is added to the table (or removed). That's why you want to refresh the metadata, to stop SSIS from crashing.

Using a SQL statement will not cause SSIS to crash, unless you changed datatypes. My reply was just an extension of your post to illustrate how one should refresh the metadata if a SQL statement is used.

Saying select * is OK because you need all columns is a valid point, but you cannot know now if at some point in the future a column is added that you don't need. For example a creation_datetime or a delete_indicator column.

Finally, as I said in my previous post, there is a difference under the hood between selecting the table from the dropdown and using select *. See here for more info. On most lists of SSIS best practices, you will find an item that advices you against using the dropdown in OLE DB sources.