Remove unnecessary catalog access check in FileBasedSystemAccessControl

[takezoe]

Description

FileBasedSystemAccessControl.checkCanSelectFromColumns() checks catalog read access but no need to check there because it's checked in AccessControlManager.

trino/core/trino-main/src/main/java/io/trino/security/AccessControlManager.java

Lines 1094 to 1105 in 17fcbb7

	public void checkCanSelectFromColumns(SecurityContext securityContext, QualifiedObjectName tableName, Set<String> columnNames)
	{
	requireNonNull(securityContext, "securityContext is null");
	requireNonNull(tableName, "tableName is null");
	requireNonNull(columnNames, "columnNames is null");
	checkCanAccessCatalog(securityContext, tableName.catalogName());
	systemAuthorizationCheck(control -> control.checkCanSelectFromColumns(securityContext.toSystemSecurityContext(), tableName.asCatalogSchemaTableName(), columnNames));
	catalogAuthorizationCheck(tableName.catalogName(), securityContext, (control, context) -> control.checkCanSelectFromColumns(context, tableName.asSchemaTableName(), columnNames));
	}

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)