We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Current, gluten use Seq.map().exists() to determine whether the specified node exists in the plan, it is better to use Seq.exists() to simply code and improve readability https://github.com/oap-project/gluten/blob/305924a19025f5ff0a65de0d38bac9e14971ecd9/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala#L156-L165 https://github.com/oap-project/gluten/blob/305924a19025f5ff0a65de0d38bac9e14971ecd9/gluten-core/src/main/scala/io/glutenproject/extension/StrategyOverrides.scala#L160-L170
Seq.map().exists()
Seq.exists()
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Description
Current, gluten use
Seq.map().exists()
to determine whether the specified node exists in the plan, it is better to useSeq.exists()
to simply code and improve readabilityhttps://github.com/oap-project/gluten/blob/305924a19025f5ff0a65de0d38bac9e14971ecd9/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala#L156-L165
https://github.com/oap-project/gluten/blob/305924a19025f5ff0a65de0d38bac9e14971ecd9/gluten-core/src/main/scala/io/glutenproject/extension/StrategyOverrides.scala#L160-L170
The text was updated successfully, but these errors were encountered: