KKcorps commented on code in PR #15657:
URL: https://github.com/apache/pinot/pull/15657#discussion_r2062983829
##
pinot-controller/src/main/resources/app/pages/TenantDetails.tsx:
##
@@ -279,6 +304,38 @@ const TenantPageDetails = ({ match }:
RouteComponentProps) => {
useEffec
KKcorps commented on PR #15641:
URL: https://github.com/apache/pinot/pull/15641#issuecomment-2834127750
@sajjad-moradi wanted your opinion on if this won't cause any backward
incompatibility issues?
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
ankitsultana opened a new pull request, #15658:
URL: https://github.com/apache/pinot/pull/15658
Does NOT touch any existing multistage code path and only contains changes
to the new optimizer.
TODO
--
This is an automated message from the Apache Git Service.
To respond to the messa
Copilot commented on code in PR #15657:
URL: https://github.com/apache/pinot/pull/15657#discussion_r2062943845
##
pinot-controller/src/main/resources/app/pages/TenantDetails.tsx:
##
@@ -279,6 +304,38 @@ const TenantPageDetails = ({ match }:
RouteComponentProps) => {
useEffec
KKcorps opened a new pull request, #15657:
URL: https://github.com/apache/pinot/pull/15657
This pull request introduces a new feature to manage the pause and resume
functionality for real-time table consumption in the Pinot Controller UI. It
includes updates to the UI components, API integr
vrajat opened a new issue, #15656:
URL: https://github.com/apache/pinot/issues/15656
The work consists of changes in `BaseSingleStageBrokerHandler` processing
logical tables to extract metadata of multiple physical tables and pass a map
of table, segment list to SSE query executor in each s
abhishekbafna commented on PR #15515:
URL: https://github.com/apache/pinot/pull/15515#issuecomment-2834051961
Closes [#15556](https://github.com/apache/pinot/issues/15556)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new c894265b1f Allow raw JSON as input for Add t
KKcorps merged PR #15647:
URL: https://github.com/apache/pinot/pull/15647
--
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: commits-unsubscr...@pinot.apac
codecov-commenter commented on PR #15655:
URL: https://github.com/apache/pinot/pull/15655#issuecomment-2833993457
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15655?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
vrajat commented on code in PR #15515:
URL: https://github.com/apache/pinot/pull/15515#discussion_r2062879617
##
pinot-common/src/main/java/org/apache/pinot/common/config/provider/TableCache.java:
##
@@ -144,6 +172,17 @@ public String getActualTableName(String tableName) {
praveenc7 commented on code in PR #15350:
URL: https://github.com/apache/pinot/pull/15350#discussion_r2062747309
##
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/merger/SelectionOnlyResultsBlockMerger.java:
##
@@ -46,15 +50,42 @@ public void mergeResultsBlocks(
praveenc7 commented on PR #15350:
URL: https://github.com/apache/pinot/pull/15350#issuecomment-2833691720
> How is this handled on the broker when the DataTable schema mismatches
between 2 servers for a select * query?
Added this support as well
--
This is an automated message from
praveenc7 commented on code in PR #15350:
URL: https://github.com/apache/pinot/pull/15350#discussion_r2062747109
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java:
##
@@ -1729,6 +1729,7 @@ static void updateColumnNames(
codecov-commenter commented on PR #15654:
URL: https://github.com/apache/pinot/pull/15654#issuecomment-2833477223
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15654?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
xiangfu0 merged PR #15643:
URL: https://github.com/apache/pinot/pull/15643
--
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: commits-unsubscr...@pinot.apa
Copilot commented on code in PR #15654:
URL: https://github.com/apache/pinot/pull/15654#discussion_r2062645895
##
pinot-plugins/pinot-input-format/pinot-avro-base/src/main/java/org/apache/pinot/plugin/inputformat/avro/AvroSchemaUtil.java:
##
@@ -193,8 +194,26 @@ private static O
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new adc3d79e81 Fix JsonExtractIndex with null handli
yashmayya commented on PR #15632:
URL: https://github.com/apache/pinot/pull/15632#issuecomment-2833526153
Updating ORC to the latest version (`2.1.1`) doesn't fix the issue either.
> or we could exclude ORC from the main build or package
The issue here is that the test setup in
Copilot commented on code in PR #15624:
URL: https://github.com/apache/pinot/pull/15624#discussion_r2062666847
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/SchemaConformingTransformer.java:
##
@@ -406,25 +408,25 @@ public GenericRow trans
vinodbaba commented on issue #15454:
URL: https://github.com/apache/pinot/issues/15454#issuecomment-2833519660
@xiangfu0 Could you review the
[PR](https://github.com/apache/pinot/pull/15655)
Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message,
vinodbaba opened a new pull request, #15655:
URL: https://github.com/apache/pinot/pull/15655
PR for [15454](https://github.com/apache/pinot/issues/15454)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
xiangfu0 merged PR #15616:
URL: https://github.com/apache/pinot/pull/15616
--
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: commits-unsubscr...@pinot.apa
xiangfu0 commented on PR #15626:
URL: https://github.com/apache/pinot/pull/15626#issuecomment-2833512787
in this case, I was thinking we should just let latest code to build on top
of the SPIs from the previous 2 released versions.
Just it cannot be simply by importing the old `pinot-spi`
xiangfu0 commented on PR #15632:
URL: https://github.com/apache/pinot/pull/15632#issuecomment-2833511189
Is there a plan to upgrade ORC? or we could exclude ORC from the main build?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
xiangfu0 closed issue #15516: JSON_EXTRACT_INDEX null pointer exception,
this._arguments is null
URL: https://github.com/apache/pinot/issues/15516
--
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
Copilot commented on code in PR #15641:
URL: https://github.com/apache/pinot/pull/15641#discussion_r2062653220
##
pinot-plugins/pinot-stream-ingestion/pinot-kafka-3.0/src/main/java/org/apache/pinot/plugin/stream/kafka30/KafkaStreamMetadataProvider.java:
##
@@ -97,49 +100,51 @@ p
xiangfu0 merged PR #15644:
URL: https://github.com/apache/pinot/pull/15644
--
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: commits-unsubscr...@pinot.apa
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new b7887e2545 Bump com.fasterxml.jackson:jackson-bo
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
dependabot/maven/com.fasterxml.jackson-jackson-bom-2.19.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was cd6b57fafc Bump com.fasterxml.jackson:jackson-bom from 2.18.3 to 2
xiangfu0 merged PR #15642:
URL: https://github.com/apache/pinot/pull/15642
--
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: commits-unsubscr...@pinot.apa
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.31.29
in repository https://gitbox.apache.org/repos/asf/pinot.git
was ba592e751b Bump software.amazon.awssdk:bom from 2.31.28 to 2.31.29
The
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d1d265128c Bump org.apache.httpcomponents.client
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 1a5bb32fe7 Bump software.amazon.awssdk:bom from
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
dependabot/maven/org.apache.httpcomponents.client5-httpclient5-5.4.4
in repository https://gitbox.apache.org/repos/asf/pinot.git
was ae11ac0195 Bump org.apache.httpcomponents.client5:httpc
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
dependabot/maven/com.dynatrace.hash4j-hash4j-0.22.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 771c16f0ce Bump com.dynatrace.hash4j:hash4j from 0.21.0 to 0.22.0
The r
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 489e9927ef Bump com.dynatrace.hash4j:hash4j from
xiangfu0 merged PR #15645:
URL: https://github.com/apache/pinot/pull/15645
--
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: commits-unsubscr...@pinot.apa
destrex271 commented on PR #15653:
URL: https://github.com/apache/pinot/pull/15653#issuecomment-2833498340
Also @xiangfu0 , my initial idea for this is that the current bash script
will run inside Github Actions and then raise a pull request with the updated
License binary file. Does that a
destrex271 commented on PR #15653:
URL: https://github.com/apache/pinot/pull/15653#issuecomment-2833460969
> Thanks for submitting the PR! Could you also paste a sample output or
changes ?
Sure! I am currently figuring out how to validate licenses of all the
dependencies. Once that i
xiangfu0 opened a new pull request, #15654:
URL: https://github.com/apache/pinot/pull/15654
seems better to have this config enabled to support types like bigDecimal etc
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
xiangfu0 commented on PR #15653:
URL: https://github.com/apache/pinot/pull/15653#issuecomment-2833448389
Thanks for submitting the PR!
Could you also paste a sample output or changes ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
xiangfu0 merged PR #15625:
URL: https://github.com/apache/pinot/pull/15625
--
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: commits-unsubscr...@pinot.apa
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new fd802fcf9a Add netty memory related metrics for
destrex271 opened a new pull request, #15653:
URL: https://github.com/apache/pinot/pull/15653
Hi,
This PR is to resolve issue:
- #15460
**Changes**
- Add a new Github Actions workflow which is triggered on relevant
dependabot PRs
- Adds a new bash script to run all the
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 43089b15f3 Add netty memory related metrics for
xiangfu0 merged PR #15651:
URL: https://github.com/apache/pinot/pull/15651
--
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: commits-unsubscr...@pinot.apa
Copilot commented on code in PR #15651:
URL: https://github.com/apache/pinot/pull/15651#discussion_r2062502414
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/channel/GrpcMailboxServer.java:
##
@@ -49,24 +55,48 @@ public class GrpcMailboxServer extends
Pinot
ishaan1091 commented on issue #15311:
URL: https://github.com/apache/pinot/issues/15311#issuecomment-2833253827
Sure, Let me take a look at the linked resources once I get some time this
week post which I will try to raise a PR
--
This is an automated message from the Apache Git Service.
49 matches
Mail list logo
