GitHub user FelixYBW added a comment to the discussion: Backend-specific contents in common code spaces
@zhztheplayer Can you fill the table? Does all the gluten- folder hold CK and VL code? in the code do we have lots of `if CK xxx elif VL yyy`? We should avoid to use the `if else` in code. I remember we have cleaned once before. Instead to put them in different folder, like ep/build-clickhouse and ep/build-velox | CK code | VL code | Java | CPP | comments -- | -- | -- | -- | -- | -- backends-clickhouse | Y | | Y | | backends-velox | | Y | Y | | cpp-ch | Y | | | Y | cpp | | Y | | Y | dev | | | | | docs | | | | | ep | Y | Y | | Y | gluten-arrow | | | | | gluten-celeborn | | | | | gluten-core | | | | | gluten-delta | | | | | gluten-hudi | | | | | gluten-iceberg | | | | | gluten-ras | | | | | gluten-substrait | | | | | gluten-ui | | | | | gluten-uniffle | | | | | gluten-ut | | | | | package | | | | | shims | | | | | tools | | | | | GitHub link: https://github.com/apache/incubator-gluten/discussions/7735#discussioncomment-11106811 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
