Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GLUTEN-7325] [CH] add a config to enable turn off read json #7333

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

shuai-xu
Copy link
Contributor

What changes were proposed in this pull request?

For json format, it's difficult to get exactly same result with vanila spark, so add a config to enable turn off native read json.

(Fixes: #7325)

How was this patch tested?

This patch was tested by manual tests.

Copy link

#7325

Copy link

Run Gluten Clickhouse CI

@@ -130,6 +130,12 @@ object CHBackendSettings extends BackendSettingsApi with Logging {

val GLUTEN_AQE_PROPAGATEEMPTY: String = CHConf.prefixOf("aqe.propagate.empty.relation")

val GLUTEN_ALLOW_READ_JSON: String =
GlutenConfig.GLUTEN_CONFIG_PREFIX + CHBackend.BACKEND_NAME +
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use CHConf.runtimeSettings instead of GlutenConfig.GLUTEN_CONFIG_PREFIX + CHBackend.BACKEND_NAME+".runtime_settings"

@baibaichen
Copy link
Contributor

@shuai-xu please add a ut

Copy link

Run Gluten Clickhouse CI

3 similar comments
Copy link

Run Gluten Clickhouse CI

@shuai-xu
Copy link
Contributor Author

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Oct 8, 2024

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Oct 8, 2024

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Oct 8, 2024

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Oct 8, 2024

Run Gluten Clickhouse CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CH] read result may diff with spark for json format
2 participants