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-6887][VL] Daily Update Velox Version (2025_01_23) #8594

Merged
merged 4 commits into from
Jan 24, 2025

Conversation

GlutenPerfBot
Copy link
Contributor

Upstream Velox's New Commits:

b9bd4917f by Leonid Chistov, fix: Fix operator addInput stats collection (11960)
58ff4ac87 by HolyLow, fix: regexp_extract returns match in mismatched group (12109)
ae34c2983 by aditi-pandit, fix: Initialize the Memory Manager in RemoteFunctionServiceMain (12095)
d7e1f5a07 by Daniel Hunte, fix(fuzzer): Remove hardcoded table names in Fuzzers (12120)
9fd0a380a by wangguangxin.cn, feat: Support UNKNOWN type for Spark collect_set aggregate function (12013)
8480a6073 by Christian Zentgraf, refactor(build): Separate thrift and arrow install on Linux (12034)
05fc7f8e3 by Chengcheng Jin, feat: Support read local file async (11869)
9f4d43df4 by Pedro Eugenio Rocha Pedreira, fix(merge-join): Properly load lazies on right join (12141)
13f2c7c4e by Jacob Wujciak-Jens, build(ci): Collect build metrics for monolithic library (12140)
1426473ed by Sergey Pershin, feat: Add MAP() signature for MapFunction. (12115)
523bf82c2 by duanmeng, misc: Enable trace tool test (12139)
d929a348a by Zac Wen, feat: Support o_direct flag for read file (12138)
f81a8c4f3 by duanmeng, fix: Flaky test FilterProjectReplayerTest.projectOnly (12124)

Signed-off-by: glutenperfbot <glutenperfbot@intel-internal.com>
Copy link

#6887

@zhouyuan
Copy link
Contributor

failure seems related with: facebookincubator/velox#12109

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
@github-actions github-actions bot added the INFRA label Jan 23, 2025
Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
@FelixYBW
Copy link
Contributor

facebookincubator/velox#12109

We still have some result mismatch issues related to re2. Can we add it to exception? Is the use case common?

@zhouyuan zhouyuan merged commit 0e8eb80 into apache:main Jan 24, 2025
46 checks passed
@zhouyuan
Copy link
Contributor

facebookincubator/velox#12109

We still have some result mismatch issues related to re2. Can we add it to exception? Is the use case common?

Yes, I reverted this commit in oap/Velox temporary and we will make a new impl for Spark-sql in Velox
Cc: @PHILO-HE

Thanks, -yuan

@PHILO-HE
Copy link
Contributor

@FelixYBW, @zhouyuan, this is a case that Presto & Spark have different handling. Just created on pr to fix it: facebookincubator/velox#12162.

baibaichen pushed a commit to baibaichen/gluten that referenced this pull request Feb 1, 2025
Upstream Velox's New Commits:

b9bd4917f by Leonid Chistov, fix: Fix operator addInput stats collection (11960)
58ff4ac87 by HolyLow, fix: regexp_extract returns match in mismatched group (12109)
ae34c2983 by aditi-pandit, fix: Initialize the Memory Manager in RemoteFunctionServiceMain (12095)
d7e1f5a07 by Daniel Hunte, fix(fuzzer): Remove hardcoded table names in Fuzzers (12120)
9fd0a380a by wangguangxin.cn, feat: Support UNKNOWN type for Spark collect_set aggregate function (12013)
8480a6073 by Christian Zentgraf, refactor(build): Separate thrift and arrow install on Linux (12034)
05fc7f8e3 by Chengcheng Jin, feat: Support read local file async (11869)
9f4d43df4 by Pedro Eugenio Rocha Pedreira, fix(merge-join): Properly load lazies on right join (12141)
13f2c7c4e by Jacob Wujciak-Jens, build(ci): Collect build metrics for monolithic library (12140)
1426473ed by Sergey Pershin, feat: Add MAP() signature for MapFunction. (12115)
523bf82c2 by duanmeng, misc: Enable trace tool test (12139)
d929a348a by Zac Wen, feat: Support o_direct flag for read file (12138)
f81a8c4f3 by duanmeng, fix: Flaky test FilterProjectReplayerTest.projectOnly (12124)
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.

4 participants