[VL] Remove resolving ViewFs file path from scan validation

[PHILO-HE]

What changes were proposed in this pull request?

I note Velox's HDFS filesystem allows ViewFs scheme.
https://github.com/facebookincubator/velox/blob/main/velox/connectors/hive/storage_adapters/hdfs/HdfsFileSystem.cpp#L112.

Then, we can just rely on the uniform JNI call through allSupportedByRegisteredFileSystems, instead of resolving the ViewFS file path in Gluten scan validation.

How was this patch tested?

Customer validation.

[github-actions]

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

• Other pull requests

[PHILO-HE]

@wangyum, @JkSelf, could you take a look?

[JkSelf]

@PHILO-HE We still require this support on the Gluten side. Velox's ViewFS is only supported on a single node and does not extend to cross-node clusters, making it unsuitable for @wangyum environment.

[PHILO-HE]

@PHILO-HE We still require this support on the Gluten side. Velox's ViewFS is only supported on a single node and does not extend to cross-node clusters, making it unsuitable for @wangyum environment.

@JkSelf, I understand Velox doesn't fully support ViewFS, so path resolving is required before plan execution. But during scan validation, as ViewFS path will always be supported by HDFS file system if it's registered (it's guaranteed by allSupportedByRegisteredFileSystems), it seems unnecessary to do a path resolving here.

[JkSelf]

@PHILO-HE Oh, I understand. It makes sense to remove this conversion during the validation phase. Thanks.

[JkSelf]

LGTM.

[wangyum]

lgmt.