[VL] Avoid reapplying code patches for external projects when ENABLE_EP_CACHE=ON

[zhztheplayer]

This is to fix error "Failed to apply compilation fixes to Velox: ..." when running build scripts with ENABLE_EP_CACHE=ON.

Also, remove option SKIP_BUILD_EP since its functionality overlaps ENABLE_EP_CACHE. One can use ENABLE_EP_CACHE=ON to skip building external projects which were already built earlier.

[github-actions]

Thanks for opening a pull request!

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

https://github.com/oap-project/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

[zhouyuan]

will this also check the compile flags?
e.g. if velox commit does not change, but the compile flags are modified, --enable_hdfs=ON/OFF

[zhztheplayer]

will this also check the compile flags?
e.g. if velox commit does not change, but the compile flags are modified, --enable_hdfs=ON/OFF

No. So the script should be improved then to support that. Currently only commit hash be used as the cache key.

[zhztheplayer]

@zhouyuan @PHILO-HE @rui-mo

[PHILO-HE]

Looks good! Thanks!

[GlutenPerfBot]

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query	log/native_3916_time.csv	log/native_master_12_04_2023_94c91c55d_time.csv	difference	percentage
q1	33.05	34.65	1.599	104.84%
q2	26.32	24.91	-1.413	94.63%
q3	37.92	36.38	-1.533	95.96%
q4	37.76	37.37	-0.390	98.97%
q5	71.63	72.63	0.997	101.39%
q6	6.95	6.87	-0.081	98.83%
q7	84.58	85.44	0.859	101.02%
q8	85.55	87.86	2.317	102.71%
q9	125.35	124.52	-0.833	99.34%
q10	46.75	46.04	-0.710	98.48%
q11	20.14	20.12	-0.011	99.94%
q12	24.95	26.71	1.753	107.02%
q13	45.33	46.45	1.118	102.47%
q14	18.29	14.55	-3.746	79.52%
q15	29.49	28.16	-1.330	95.49%
q16	15.88	15.75	-0.126	99.21%
q17	102.09	103.10	1.012	100.99%
q18	150.09	150.63	0.547	100.36%
q19	13.70	12.90	-0.797	94.18%
q20	27.86	27.73	-0.130	99.53%
q21	222.53	222.80	0.264	100.12%
q22	13.41	13.06	-0.354	97.36%
total	1239.62	1238.63	-0.990	99.92%