Re: [PR] [GOBBLIN-2088] Add retry to OH replication final catalog commit [gobblin]

2024-06-21 Thread via GitHub
phet commented on code in PR #3976: URL: https://github.com/apache/gobblin/pull/3976#discussion_r1648405152 ## gobblin-data-management/src/main/java/org/apache/gobblin/data/management/copy/iceberg/IcebergRegisterStep.java: ## @@ -54,7 +66,12 @@ public class IcebergRegisterStep

Re: [PR] GH-34785: [C++][Parquet] Parquet Bloom Filter Writer Implementation [arrow]

2024-06-21 Thread via GitHub
mapleFU commented on code in PR #37400: URL: https://github.com/apache/arrow/pull/37400#discussion_r1648418819 ## cpp/src/parquet/bloom_filter_builder.cc: ## @@ -0,0 +1,158 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

[PR] CAMEL-20903: camel-jbang - Only load custom plugin if in use [camel]

2024-06-21 Thread via GitHub
davsclaus opened a new pull request, #14612: URL: https://github.com/apache/camel/pull/14612 # Description # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) #

Re: [PR] [fix](map) fix core dump when upgrading from 1.2.x to 2.0.x with map datatype column [doris]

2024-06-21 Thread via GitHub
amorynan commented on PR #36635: URL: https://github.com/apache/doris/pull/36635#issuecomment-2182065252 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] GH-34785: [C++][Parquet] Parquet Bloom Filter Writer Implementation [arrow]

2024-06-21 Thread via GitHub
emkornfield commented on code in PR #37400: URL: https://github.com/apache/arrow/pull/37400#discussion_r1648419112 ## cpp/src/parquet/bloom_filter_builder.cc: ## @@ -0,0 +1,158 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

Re: [PR] fix: default logging [superset]

2024-06-21 Thread via GitHub
jessie-ross commented on code in PR #2: URL: https://github.com/apache/superset/pull/2#discussion_r1648418971 ## superset/utils/logging_configurator.py: ## @@ -41,19 +41,11 @@ def configure_logging( if app_config["SILENCE_FAB"]:

Re: [PR] [VL] Fix potential double unload of JNI libraries on Java 11+ [incubator-gluten]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #6146: URL: https://github.com/apache/incubator-gluten/pull/6146#issuecomment-2182064562 Run Gluten Clickhouse CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] Emit metrics for S3UploadThreadPool (druid)

2024-06-21 Thread via GitHub
cryptoe commented on PR #16616: URL: https://github.com/apache/druid/pull/16616#issuecomment-2182064696 The task failures are unrelated. Going ahead with merge. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [fix](be) Check MD5 of the downloaded files before ingesting binlog [doris]

2024-06-21 Thread via GitHub
dataroaring merged PR #36621: URL: https://github.com/apache/doris/pull/36621 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [fix](txn insert) Fix txn insert connect to observer fe in cloud mode [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36615: URL: https://github.com/apache/doris/pull/36615#issuecomment-2182052374 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [enhance](mtmv)support partition tvf [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36479: URL: https://github.com/apache/doris/pull/36479#issuecomment-2182062011 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [improvement](meta) serialize meta via gson part3 [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36552: URL: https://github.com/apache/doris/pull/36552#issuecomment-2182049199 ClickBench: Total hot run time: 30.11 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] RESTful API supports killing engine forcibly [kyuubi]

2024-06-21 Thread via GitHub
pan3793 commented on PR #6008: URL: https://github.com/apache/kyuubi/pull/6008#issuecomment-2182045326 @turboFei @zhaohehuhu Sorry for forgetting this one, the new approach lgtm, let fix the UT the get it in. -- This is an automated message from the Apache Git Service. To respond to the

[PR] [Feature][influxdb] Support multi-table source feature [seatunnel]

2024-06-21 Thread via GitHub
531651225 opened a new pull request, #7039: URL: https://github.com/apache/seatunnel/pull/7039 ### Purpose of this pull request refer to https://github.com/apache/seatunnel/issues/6925 ### Does this PR introduce _any_ user-facing change? ### How was

Re: [PR] [DNR] test [incubator-gluten]

2024-06-21 Thread via GitHub
zhli1142015 commented on PR #6171: URL: https://github.com/apache/incubator-gluten/pull/6171#issuecomment-2182044777 /Benchmark TPCH -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] HDDS-10517. Recon - Add a UI component for showing DN decommissioning detailed status and info [ozone]

2024-06-21 Thread via GitHub
smitajoshi12 commented on PR #6724: URL: https://github.com/apache/ozone/pull/6724#issuecomment-2182061047 @dombizita @devabhishekpal Hi Zita and Abhishek Could you review this PR -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] Uniformly use [...] to replace the phrase [More]. [hertzbeat]

2024-06-21 Thread via GitHub
kerwin612 opened a new pull request, #2115: URL: https://github.com/apache/hertzbeat/pull/2115 ## What's changed? Uniformly use [...] to replace the phrase [More]. ## Checklist - [x] I have read the [Contributing

Re: [I] How to implement hint sharding algorithm? [shardingsphere]

2024-06-21 Thread via GitHub
songjiavc commented on issue #25985: URL: https://github.com/apache/shardingsphere/issues/25985#issuecomment-2182056164 This issue is finished? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] [improvement](meta) serialize meta via gson part3 [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36552: URL: https://github.com/apache/doris/pull/36552#issuecomment-2182043605 TPC-DS: Total hot run time: 172873 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [doc](arrow-flight-sql) Add arrow flight sql docs [doris-website]

2024-06-21 Thread via GitHub
xinyiZzz commented on PR #776: URL: https://github.com/apache/doris-website/pull/776#issuecomment-2182077179 > Please also add the file in sidebars.json & version-2.1-sidebars.json & version-2.1.json done, version-2.1.json seem not need modify -- This is an automated message from

Re: [PR] Bump com.netflix.nebula.ospackage from 11.9.0 to 11.9.1 [ignite-3]

2024-06-21 Thread via GitHub
ptupitsyn merged PR #3840: URL: https://github.com/apache/ignite-3/pull/3840 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [fix] fix ci && Dubboctl deploy subcommand error fix [dubbo-kubernetes]

2024-06-21 Thread via GitHub
chickenlj merged PR #304: URL: https://github.com/apache/dubbo-kubernetes/pull/304 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [I] apisix will retry the post method? [apisix]

2024-06-21 Thread via GitHub
kellyseeme commented on issue #11359: URL: https://github.com/apache/apisix/issues/11359#issuecomment-2182056143 > Do you mean you want APISIX to return 5xx instead of trying the next upstream node? yes,which place i can cancle this settings? -- This is an automated message from

Re: [I] [multi-stage] [stability] QueryDispatcher Grpc needs to have a deadline and should be async [pinot]

2024-06-21 Thread via GitHub
Jackie-Jiang closed issue #9671: [multi-stage] [stability] QueryDispatcher Grpc needs to have a deadline and should be async URL: https://github.com/apache/pinot/issues/9671 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [I] Support passing a slot range in CLUSTERX MIGRATE [kvrocks]

2024-06-21 Thread via GitHub
PokIsemaine commented on issue #2355: URL: https://github.com/apache/kvrocks/issues/2355#issuecomment-2182077655 First of all, I would like to discuss the use of commands, and noting that `CLUSTERX SETSLOT` has similar needs, I decided to refer to the following PR and issues.

Re: [I] [Feature][Manager] Restrict sortTaskName and sortConsumeName the same with datanodeName when migrate Inlong Group [inlong]

2024-06-21 Thread via GitHub
vernedeng closed issue #10474: [Feature][Manager] Restrict sortTaskName and sortConsumeName the same with datanodeName when migrate Inlong Group URL: https://github.com/apache/inlong/issues/10474 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] MAPREDUCE-7472. Decode value of hive.query.string for the job Confguration which was encoded by hive [hadoop]

2024-06-21 Thread via GitHub
hadoop-yetus commented on PR #6520: URL: https://github.com/apache/hadoop/pull/6520#issuecomment-2182048365 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] GH-42193: [Java] Update dependency to maintain JUnit 5 only [arrow]

2024-06-21 Thread via GitHub
lidavidm merged PR #42206: URL: https://github.com/apache/arrow/pull/42206 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [feat](nereids)support partition level column stats [doris]

2024-06-21 Thread via GitHub
englefly commented on PR #35875: URL: https://github.com/apache/doris/pull/35875#issuecomment-2182075777 run p0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] WW-5430 Uses Freemarker native support for JakartaEE instead of manually transforming the artifacts [struts]

2024-06-21 Thread via GitHub
lukaszlenart opened a new pull request, #970: URL: https://github.com/apache/struts/pull/970 Closes [WW-5430](https://issues.apache.org/jira/browse/WW-5430) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] [YUNIKORN-2686] Validate user and group specified in filter config [yunikorn-core]

2024-06-21 Thread via GitHub
manirajv06 opened a new pull request, #898: URL: https://github.com/apache/yunikorn-core/pull/898 ### What is this PR for? Enhanced the user and group filter tests based on new user validation characters. ### What type of PR is it? * [ ] - Improvement ### Todos * [ ]

Re: [PR] [feature](function) support new function replace_empty (#36283) [doris]

2024-06-21 Thread via GitHub
morningman commented on PR #36656: URL: https://github.com/apache/doris/pull/36656#issuecomment-2182074432 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [improvement](meta) serialize meta via gson part3 [doris]

2024-06-21 Thread via GitHub
dataroaring commented on PR #36552: URL: https://github.com/apache/doris/pull/36552#issuecomment-2182048549 run feut -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [Build] Unify bnd-maven-plugin and maven-jar-plugin invocations [maven-resolver]

2024-06-21 Thread via GitHub
gnodet commented on PR #509: URL: https://github.com/apache/maven-resolver/pull/509#issuecomment-2182072618 @HannesWell could you create a similar PR for master please ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] [Oracle][MySQL][SqlServer][PostgresSQL] Fix Oracle/MySQL/SqlServer/PostgresSQL CDC parser schema change event failed [flink-cdc]

2024-06-21 Thread via GitHub
mdzz9527 commented on PR #2315: URL: https://github.com/apache/flink-cdc/pull/2315#issuecomment-2182073190 合并到哪个版本了呢 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] helm chart is not picking security context value passed in values.yml [couchdb-helm]

2024-06-21 Thread via GitHub
AnishaM7 commented on issue #159: URL: https://github.com/apache/couchdb-helm/issues/159#issuecomment-218204 yes, we are using security as you mentioned, but we need to add these value in values.yaml file ``` # -- the initial number of nodes in the CouchDB cluster. clusterSize:

Re: [PR] [improvement](meta) serialize meta via gson part3 [doris]

2024-06-21 Thread via GitHub
dataroaring commented on PR #36552: URL: https://github.com/apache/doris/pull/36552#issuecomment-2182072400 run feut -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Fix several bugs [skywalking-banyandb]

2024-06-21 Thread via GitHub
wu-sheng merged PR #473: URL: https://github.com/apache/skywalking-banyandb/pull/473 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Fix several bugs [skywalking-banyandb]

2024-06-21 Thread via GitHub
wu-sheng commented on PR #473: URL: https://github.com/apache/skywalking-banyandb/pull/473#issuecomment-2182072118 Let's update the demo and see how this is going. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [hotfix][streaming] Move the TestProcessingTimeService.java down to the corresponding test package. [flink]

2024-06-21 Thread via GitHub
RocMarshal commented on PR #24965: URL: https://github.com/apache/flink/pull/24965#issuecomment-2182071760 @flinkbot run azure -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] [CH] support Levenshtein distance [incubator-gluten]

2024-06-21 Thread via GitHub
zzcclp merged PR #6108: URL: https://github.com/apache/incubator-gluten/pull/6108 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] CAMEL-20895: camel-djl - The image classification predictors should not round the resulted probabilities [camel]

2024-06-21 Thread via GitHub
tadayosi commented on PR #14611: URL: https://github.com/apache/camel/pull/14611#issuecomment-2182031570 Hmm, the test failures seem to be unrelated to the pull req. It seems there are some other commits that have left uncommitted changes? -- This is an automated message from the Apache

Re: [I] values.yaml `dags.gitSync.containerLifecycleHooks` Does not working [airflow]

2024-06-21 Thread via GitHub
boring-cyborg[bot] commented on issue #40364: URL: https://github.com/apache/airflow/issues/40364#issuecomment-2182070360 Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for

Re: [PR] PHOENIX-7318 : Support JSON_MODIFY in Upserts [phoenix]

2024-06-21 Thread via GitHub
ranganathg commented on code in PR #1904: URL: https://github.com/apache/phoenix/pull/1904#discussion_r1648442847 ## phoenix-core-client/src/main/java/org/apache/phoenix/compile/UpsertCompiler.java: ## @@ -842,11 +851,27 @@ public MutationPlan compile(UpsertStatement upsert)

Re: [PR] [Fix](export/outfile) Support compression when exporting data to Parquet / ORC. [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36490: URL: https://github.com/apache/doris/pull/36490#issuecomment-2182018700 TeamCity be ut coverage result: Function Coverage: 36.46% (9006/24699) Line Coverage: 28.01% (73864/263707) Region Coverage: 27.48% (38359/139584) Branch Coverage:

Re: [PR] [HUDI-7849] Reduce time spent on running testFiltersInFileFormat [hudi]

2024-06-21 Thread via GitHub
hudi-bot commented on PR #11423: URL: https://github.com/apache/hudi/pull/11423#issuecomment-2182017381 ## CI report: * 42afc3dd0a30cf1d33c325fbb74bb42759830ed9 Azure:

Re: [I] Implement communication with Dubbo Java and Dubbo Go [dubbo-js]

2024-06-21 Thread via GitHub
thegenius commented on issue #363: URL: https://github.com/apache/dubbo-js/issues/363#issuecomment-2182067941 我尝试了一下用dubbo-js连接dubbo-java, 发现Server received greet request null,是底层还有什么不支持的东西吗 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] WIP [UPGRADE] MAVEN dependencies [james-project]

2024-06-21 Thread via GitHub
chibenwa commented on code in PR #2302: URL: https://github.com/apache/james-project/pull/2302#discussion_r1648446629 ##

Re: [PR] feat(c++,spark): support json payload file format [incubator-graphar]

2024-06-21 Thread via GitHub
acezen commented on PR #518: URL: https://github.com/apache/incubator-graphar/pull/518#issuecomment-2182068360 > after making the changes when i tried to push git showed me some conflicts and after gpt + google search i followed the below > > git stash git pull origin

Re: [PR] HDDS-10946. Test combinations of rename and rewrite [ozone]

2024-06-21 Thread via GitHub
adoroszlai commented on PR #6823: URL: https://github.com/apache/ozone/pull/6823#issuecomment-2182026994 Thanks @fapifta, @kerneltime, @sodonnel for the review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [I] Quick Start with kvrocks cluster [kvrocks-controller]

2024-06-21 Thread via GitHub
PokIsemaine closed issue #185: Quick Start with kvrocks cluster URL: https://github.com/apache/kvrocks-controller/issues/185 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] JAMES-2586 Optimize findNonPersonalMailboxes method in PostgresMailboxMapper [james-project]

2024-06-21 Thread via GitHub
chibenwa commented on code in PR #2296: URL: https://github.com/apache/james-project/pull/2296#discussion_r1648441376 ## backends-common/postgres/src/main/java/org/apache/james/backends/postgres/PostgresTable.java: ## @@ -52,11 +53,58 @@ default FinalStage

Re: [PR] Kafka Connect: Add kerberos authentication option [iceberg]

2024-06-21 Thread via GitHub
Dawnpool commented on PR #10173: URL: https://github.com/apache/iceberg/pull/10173#issuecomment-2182024037 @bryanck If I got it right, you’re asking if we can add those configs for Kerberos authentication via Hadoop config. I found there are Kerberos options for service principals like

Re: [PR] Change remap ACL matching policy to 'FIRST_EXPLICIT_MATCH_WINS' [trafficserver]

2024-06-21 Thread via GitHub
bneradt commented on code in PR #11433: URL: https://github.com/apache/trafficserver/pull/11433#discussion_r1648410926 ## tests/gold_tests/ip_allow/ip_category.test.py: ## @@ -230,6 +230,7 @@ def _configure_traffic_server(self) -> None:

Re: [PR] Flink: Fix duplicate data in Flink's upsert writer for format V2 [iceberg]

2024-06-21 Thread via GitHub
pvary commented on code in PR #10526: URL: https://github.com/apache/iceberg/pull/10526#discussion_r1648420475 ## flink/v1.19/flink/src/test/java/org/apache/iceberg/flink/sink/TestIcebergFilesCommitter.java: ## @@ -887,6 +921,55 @@ public void

Re: [PR] Emit metrics for S3UploadThreadPool (druid)

2024-06-21 Thread via GitHub
cryptoe merged PR #16616: URL: https://github.com/apache/druid/pull/16616 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [fix](planner) fix result wrong caused by mixed use multiple columns with type of datetime/datetimev2/date/datev2 of function coalesce [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36640: URL: https://github.com/apache/doris/pull/36640#issuecomment-2182067840 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [fix](planner) fix result wrong caused by mixed use multiple columns with type of datetime/datetimev2/date/datev2 of function coalesce [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36640: URL: https://github.com/apache/doris/pull/36640#issuecomment-2182067880 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[I] 1.8.0 branchRollback failed [incubator-seata]

2024-06-21 Thread via GitHub
mrsuibian opened a new issue, #6623: URL: https://github.com/apache/incubator-seata/issues/6623 [error,61] - [stacktrace]branchRollback failed. branchType:[AT], xid:[10.19.36.125:8092:5323786182041662173], branchId:[5323786182041662175],

Re: [PR] [Chore](GA) Set the maintainer for the FE ENV file [doris]

2024-06-21 Thread via GitHub
CalvinKirs merged PR #36650: URL: https://github.com/apache/doris/pull/36650 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [VL] Fix potential double unload of JNI libraries on Java 11+ [incubator-gluten]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #6146: URL: https://github.com/apache/incubator-gluten/pull/6146#issuecomment-2182062919 Run Gluten Clickhouse CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] [Build] Unify bnd-maven-plugin and maven-jar-plugin invocations [maven-resolver]

2024-06-21 Thread via GitHub
gnodet merged PR #509: URL: https://github.com/apache/maven-resolver/pull/509 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] load-tsfile: Add retry logic for load memory controls [iotdb]

2024-06-21 Thread via GitHub
codecov[bot] commented on PR #12782: URL: https://github.com/apache/iotdb/pull/12782#issuecomment-2182021666 ## [Codecov](https://app.codecov.io/gh/apache/iotdb/pull/12782?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report

Re: [PR] test: fix test failure on ubuntu 24.04: "top: unknown option 'n'" [cloudstack]

2024-06-21 Thread via GitHub
sureshanaparti commented on PR #9262: URL: https://github.com/apache/cloudstack/pull/9262#issuecomment-2182015956 test_service_offering.py has no failures, manually verified cmds in Rocky Linux 8.4, Oracle Linux 8.6. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] [Fix](export/outfile) Support compression when exporting data to Parquet / ORC. [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36490: URL: https://github.com/apache/doris/pull/36490#issuecomment-2182020717 TPC-DS: Total hot run time: 172088 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] CAMEL-20895: camel-djl - The image classification predictors should not round the resulted probabilities [camel]

2024-06-21 Thread via GitHub
davsclaus commented on PR #14611: URL: https://github.com/apache/camel/pull/14611#issuecomment-2182061580 try to rebase as main has been fixed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] [Feature](json_functions) support json-keys [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36411: URL: https://github.com/apache/doris/pull/36411#issuecomment-2182014567 ClickBench: Total hot run time: 30.83 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [core] Check the validity of scan staring snapshotId [paimon]

2024-06-21 Thread via GitHub
LinMingQiang commented on PR #3554: URL: https://github.com/apache/paimon/pull/3554#issuecomment-2182019991 @JingsongLi Help to review this PR, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] [feat](mtmv) compute agg expressions tree cost [doris]

2024-06-21 Thread via GitHub
englefly commented on PR #36368: URL: https://github.com/apache/doris/pull/36368#issuecomment-2182061073 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] feat(c++,spark): support json payload file format [incubator-graphar]

2024-06-21 Thread via GitHub
amygbAI commented on PR #518: URL: https://github.com/apache/incubator-graphar/pull/518#issuecomment-2182058566 after making the changes when i tried to push git showed me some conflicts and after gpt + google search i followed the below git stash git pull origin

Re: [PR] fix(embedded-sdk): add accessible title to iframe [superset]

2024-06-21 Thread via GitHub
nikola-arcadis commented on PR #27017: URL: https://github.com/apache/superset/pull/27017#issuecomment-2182060909 I don't see this change is published on npmjs registry with version 0.1.0-alpha.11. How can we get the new package with this change? -- This is an automated message from the

Re: [I] bug: When I turn on kafka-logger and body-transformer at the same time, the response body conversion of body-transformer plugin will fail. [apisix]

2024-06-21 Thread via GitHub
FirstCatchYourHare commented on issue #11356: URL: https://github.com/apache/apisix/issues/11356#issuecomment-2182056956 this bug is resolved at apisix3.2.2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feat](Nereids): Optimize Sum Literal Rewriting by Excluding Single Instances [doris]

2024-06-21 Thread via GitHub
XieJiann merged PR #35559: URL: https://github.com/apache/doris/pull/35559 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[I] EXPR_MIN/MAX do not work [pinot]

2024-06-21 Thread via GitHub
mayankshriv opened a new issue, #13452: URL: https://github.com/apache/pinot/issues/13452 It seems that the `EXPR_MIN/MAX` functions do not work in both v1 and v2 query engines. Running a simple query like below errors out, even though there is no alias in the query. On debugging, I

Re: [PR] [CORE] Creates vanilla plan when the join operators fall back [incubator-gluten]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #6093: URL: https://github.com/apache/incubator-gluten/pull/6093#issuecomment-2182019655 Run Gluten Clickhouse CI -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] Ability to specify NFS mount options while adding a primary storage and modify them on a pre-existing primary storage [cloudstack]

2024-06-21 Thread via GitHub
blueorangutan commented on PR #8947: URL: https://github.com/apache/cloudstack/pull/8947#issuecomment-2182039423 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10045 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Spark: Add SparkSQLProperty to control split-size [iceberg]

2024-06-21 Thread via GitHub
szehon-ho commented on PR #10336: URL: https://github.com/apache/iceberg/pull/10336#issuecomment-2182038718 The problem with this is that it does lead to some ambiguity as to what table the config is applying to (many queries read from several tables, for example) -- This is an

Re: [PR] [fix](clone) Increase robustness for clone [doris]

2024-06-21 Thread via GitHub
dataroaring commented on PR #36642: URL: https://github.com/apache/doris/pull/36642#issuecomment-2182039478 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] Pauseless Consumption [pinot]

2024-06-21 Thread via GitHub
kishoreg commented on issue #10147: URL: https://github.com/apache/pinot/issues/10147#issuecomment-2182011009 Reviving this. I think we should go with optimistic approach as Jackie suggested. One simplification would be to enable pauseless consumption feature only in case of commit

Re: [PR] Remove all statement handler and move method to abstract statement [shardingsphere]

2024-06-21 Thread via GitHub
FlyingZC merged PR #31796: URL: https://github.com/apache/shardingsphere/pull/31796 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] GH-34785: [C++][Parquet] Parquet Bloom Filter Writer Implementation [arrow]

2024-06-21 Thread via GitHub
emkornfield commented on PR #37400: URL: https://github.com/apache/arrow/pull/37400#issuecomment-2182032386 Sorry for the delay, I didn't get a chance to review all tests in detail. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] HDDS-10517. Recon - Add a UI component for showing DN decommissioning detailed status and info [ozone]

2024-06-21 Thread via GitHub
smitajoshi12 commented on PR #6724: URL: https://github.com/apache/ozone/pull/6724#issuecomment-2182055176 > @smitajoshi12 kindly resolve the conflicts in your code. @devmadhuu Hi Devesh Resolved conflicts and merged master with all changes and did testing and pushed changes in

Re: [PR] [improvement](meta) serialize meta via gson part3 [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36552: URL: https://github.com/apache/doris/pull/36552#issuecomment-2182033366 TPC-H: Total hot run time: 39834 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch

Re: [PR] ARTEMIS-4831: use surefire default behaviour around test failure [activemq-artemis]

2024-06-21 Thread via GitHub
gemmellr commented on code in PR #4989: URL: https://github.com/apache/activemq-artemis/pull/4989#discussion_r1647531286 ## pom.xml: ## @@ -861,9 +858,7 @@ 1 true - ${testFailureIgnore}

Re: [PR] [fix](index compaction)Change index_id from int32 to int64 to avoid overflow [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36625: URL: https://github.com/apache/doris/pull/36625#issuecomment-2180617649 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit 231ca56ce513a2f6dada675015f421d452767c6d with default session variables Stream

Re: [PR] [feature](function) support new function replace_empty (#36283) [doris]

2024-06-21 Thread via GitHub
morningman commented on PR #36656: URL: https://github.com/apache/doris/pull/36656#issuecomment-2182054059 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [I] Xcode 15.0: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 11.0, but the range of supported deployment target versions is 12.0 to 17.0.99. [cordova-ios]

2024-06-21 Thread via GitHub
breautek commented on issue #1379: URL: https://github.com/apache/cordova-ios/issues/1379#issuecomment-2180613420 > why is this "closed", since following all the "current"? documentation in https://cordova.apache.org/docs/en/12.x/guide/cli/index.html causes this, and it's not "just a

[I] [Improve][Audit] Optimize Audit domain management [inlong]

2024-06-21 Thread via GitHub
doleyzi opened a new issue, #10481: URL: https://github.com/apache/inlong/issues/10481 ### Description Optimize Audit domain management ### InLong Component InLong Audit ### Are you willing to submit PR? - [X] Yes, I am willing to submit a PR! ###

Re: [PR] [fix](txn insert) Fix txn insert connect to observer fe in cloud mode [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36615: URL: https://github.com/apache/doris/pull/36615#issuecomment-2182052350 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [fix](map) fix core dump when upgrading from 1.2.x to 2.0.x with map datatype column [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36635: URL: https://github.com/apache/doris/pull/36635#issuecomment-2182054183 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [feature](function) support new function replace_empty (#36283) [doris]

2024-06-21 Thread via GitHub
morningman opened a new pull request, #36656: URL: https://github.com/apache/doris/pull/36656 #36283 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [PR] [feature](function) support new function replace_empty (#36283) [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36656: URL: https://github.com/apache/doris/pull/36656#issuecomment-2182054009 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your

Re: [PR] HBASE-28631 Show the create time of the replication peer [hbase]

2024-06-21 Thread via GitHub
1458451310 commented on code in PR #5954: URL: https://github.com/apache/hbase/pull/5954#discussion_r1647527664 ## hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/FSReplicationPeerStorage.java: ## @@ -227,6 +227,16 @@ public ReplicationPeerConfig

Re: [PR] IGNITE-21661 Add test that reset partition can use data from a node outside of stable assignments [ignite-3]

2024-06-21 Thread via GitHub
tkalkirill commented on code in PR #3945: URL: https://github.com/apache/ignite-3/pull/3945#discussion_r1647524011 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoveryReconfigurationTest.java: ## @@ -239,10 +270,10 @@ void

[PR] Log the Camel Quarkus version on startup [camel-quarkus]

2024-06-21 Thread via GitHub
jamesnetherton opened a new pull request, #6216: URL: https://github.com/apache/camel-quarkus/pull/6216 I opened this in draft to get some feedback. For micro release of Quarkus, we may end with something like this in the console logs: ``` INFO

Re: [PR] [bugfix]Rename compression parameters [doris]

2024-06-21 Thread via GitHub
doris-robot commented on PR #36627: URL: https://github.com/apache/doris/pull/36627#issuecomment-2180607899 TPC-H: Total hot run time: 40207 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch

Re: [PR] [fix](clone) Increase robustness for clone [doris]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #36642: URL: https://github.com/apache/doris/pull/36642#issuecomment-2182040084 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] Fix several bugs [skywalking-banyandb]

2024-06-21 Thread via GitHub
hanahmily opened a new pull request, #473: URL: https://github.com/apache/skywalking-banyandb/pull/473 - [x] Add a unit test to verify that the fix works. - [x] Explain briefly why the bug exists and how to fix it. - [x] Update the [`CHANGES`

Re: [PR] test yetus [hbase]

2024-06-21 Thread via GitHub
Apache-HBase commented on PR #6005: URL: https://github.com/apache/hbase/pull/6005#issuecomment-2180611609 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] CAMEL-20838: use a JUnit extension for collecting the test name when necessary [camel]

2024-06-21 Thread via GitHub
github-actions[bot] commented on PR #14609: URL: https://github.com/apache/camel/pull/14609#issuecomment-2180617435 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel

<    1   2   3   4   5   6   7   8   9   10   >