xiangfu0 merged pull request #7342:
URL: https://github.com/apache/pinot/pull/7342
--
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:
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
mayankshriv commented on issue #7344:
URL: https://github.com/apache/pinot/issues/7344#issuecomment-903043558
Tried a simple query with latest master `select * from ;` (note
semi-colon), and seems to work. @ksnijjer please also verify at your end.
--
This is an automated message from
Jackie-Jiang commented on issue #7264:
URL: https://github.com/apache/pinot/issues/7264#issuecomment-903033843
@yupeng9 Current pinot behavior will always return the available results,
and set exceptions for unexpected scenarios such as unable to reach to a
server. For now I would suggest
Jackie-Jiang commented on issue #7344:
URL: https://github.com/apache/pinot/issues/7344#issuecomment-903027509
Duplicate with #7161. Can you please try with the latest master? The UI fix
#7165 has been merged
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
snlee 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 888c961 Add segmentNameGeneratorType config to
snleee merged pull request #7346:
URL: https://github.com/apache/pinot/pull/7346
--
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:
Jackie-Jiang commented on issue #7345:
URL: https://github.com/apache/pinot/issues/7345#issuecomment-903026876
What error did you get? Since `select id, "payload.commits.distinct" from
githubEvents limit 10` work, it should be caused by the `distinct` keyword.
--
This is an automated
yupeng9 commented on issue #7264:
URL: https://github.com/apache/pinot/issues/7264#issuecomment-903026858
Can we add it as a query option (allowPartial)?
--
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
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from 9116710 7090 segmentnamegenerator accept input file parameter (#7222)
add 48c1fc0 Fix TableNotFound error
Jackie-Jiang merged pull request #7340:
URL: https://github.com/apache/pinot/pull/7340
--
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:
Jackie-Jiang commented on issue #7264:
URL: https://github.com/apache/pinot/issues/7264#issuecomment-903026216
We need to think about how to set this `partialResponse` flag. What
scenarios should be count as partial
--
This is an automated message from the Apache Git Service.
To respond
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from 5417cbb Delete metadata properly on table deletion (#7329)
add 9116710 7090 segmentnamegenerator accept input
Jackie-Jiang merged pull request #7222:
URL: https://github.com/apache/pinot/pull/7222
--
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:
This is an automated email from the ASF dual-hosted git repository.
snlee pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from b71f38a Infer the data type for LiteralTransformFunction (#7332)
add 5417cbb Delete metadata properly on table
snleee merged pull request #7329:
URL: https://github.com/apache/pinot/pull/7329
--
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:
xiangfu0 commented on a change in pull request #7222:
URL: https://github.com/apache/pinot/pull/7222#discussion_r693275732
##
File path: pinot-integration-tests/pom.xml
##
@@ -124,6 +128,74 @@
+
+
codecov-commenter edited a comment on pull request #7346:
URL: https://github.com/apache/pinot/pull/7346#issuecomment-903005029
#
codecov-commenter edited a comment on pull request #7329:
URL: https://github.com/apache/pinot/pull/7329#issuecomment-901538754
#
codecov-commenter edited a comment on pull request #7329:
URL: https://github.com/apache/pinot/pull/7329#issuecomment-901538754
#
codecov-commenter edited a comment on pull request #7226:
URL: https://github.com/apache/pinot/pull/7226#issuecomment-895520797
#
codecov-commenter edited a comment on pull request #7329:
URL: https://github.com/apache/pinot/pull/7329#issuecomment-901538754
#
Jackie-Jiang commented on issue #7338:
URL: https://github.com/apache/pinot/issues/7338#issuecomment-903013952
@mcvsubbu It should be part of the cluster config in ZK. By default the
server uses 40 threads to handle the messages.
--
This is an automated message from the Apache Git
Jackie-Jiang commented on issue #7305:
URL: https://github.com/apache/pinot/issues/7305#issuecomment-903013488
@galalen Thanks for volunteering. Please take a look at
`ZookeeperResource.java` and add payload to the `putData()` method
--
This is an automated message from the Apache Git
mayankshriv commented on issue #7328:
URL: https://github.com/apache/pinot/issues/7328#issuecomment-903013469
cc @yupeng9
--
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.
mayankshriv commented on issue #7328:
URL: https://github.com/apache/pinot/issues/7328#issuecomment-903013387
+1, we need to make this more user friendly. Apart from the issues mentioned
above, it is also confusing to the users who are trying to push to multiple
tables, as to which outDir
Jackie-Jiang commented on a change in pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#discussion_r693266380
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/periodictask/ControllerPeriodicTask.java
##
@@ -55,14 +57,25 @@
snleee commented on a change in pull request #7346:
URL: https://github.com/apache/pinot/pull/7346#discussion_r693267519
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorFramework.java
##
@@ -110,11 +113,20 @@ public
codecov-commenter edited a comment on pull request #7346:
URL: https://github.com/apache/pinot/pull/7346#issuecomment-903005029
#
This is an automated email from the ASF dual-hosted git repository.
jackie 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 b71f38a Infer the data type for
Jackie-Jiang merged pull request #7332:
URL: https://github.com/apache/pinot/pull/7332
--
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:
codecov-commenter edited a comment on pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#issuecomment-882112478
#
Jackie-Jiang commented on a change in pull request #7332:
URL: https://github.com/apache/pinot/pull/7332#discussion_r693263074
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LiteralTransformFunction.java
##
@@ -47,24 +52,43 @@
codecov-commenter edited a comment on pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#issuecomment-882112478
#
Jackie-Jiang opened a new pull request #7347:
URL: https://github.com/apache/pinot/pull/7347
## Description
Allow logical identifier (`BOOLEAN` column) in the filter during query
parsing
--
This is an automated message from the Apache Git Service.
To respond to the message, please
codecov-commenter edited a comment on pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#issuecomment-882112478
#
codecov-commenter commented on pull request #7346:
URL: https://github.com/apache/pinot/pull/7346#issuecomment-903005029
#
jonathanlocke commented on issue #6851:
URL: https://github.com/apache/pinot/issues/6851#issuecomment-903001312
I see this too
--
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
codecov-commenter edited a comment on pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#issuecomment-882112478
#
Jackie-Jiang commented on a change in pull request #7346:
URL: https://github.com/apache/pinot/pull/7346#discussion_r693255746
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorFramework.java
##
@@ -110,11 +113,20 @@
snleee opened a new pull request #7346:
URL: https://github.com/apache/pinot/pull/7346
- Add segmentNameGeneratorType to IndexingConfing
- Add SegmentNameGeneratorFactory
- Add the test
TODO: in the long term, we should have a dedicated
segment generator related config in
This is an automated email from the ASF dual-hosted git repository.
jackie 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 a074cde mark and sweep indices for V3 segment
Jackie-Jiang merged pull request #7301:
URL: https://github.com/apache/pinot/pull/7301
--
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:
Jackie-Jiang commented on a change in pull request #7332:
URL: https://github.com/apache/pinot/pull/7332#discussion_r693221321
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LiteralTransformFunction.java
##
@@ -104,7 +128,11 @@ public
Jackie-Jiang commented on a change in pull request #7332:
URL: https://github.com/apache/pinot/pull/7332#discussion_r693220587
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LiteralTransformFunction.java
##
@@ -47,24 +52,43 @@
lfeagan commented on issue #6921:
URL: https://github.com/apache/pinot/issues/6921#issuecomment-902941838
In this case, I am referring to Arrow as a specification for a binary format
to represent data, rather than an implementation. I am looking for support for
reading data as a client.
Jackie-Jiang commented on a change in pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#discussion_r693189066
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/api/resources/PinotBrokerDebug.java
##
@@ -59,7 +59,11 @@
galalen commented on issue #7305:
URL: https://github.com/apache/pinot/issues/7305#issuecomment-902932007
I would love to work on it, please point me to the class
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
mayankshriv commented on issue #7344:
URL: https://github.com/apache/pinot/issues/7344#issuecomment-902920833
Yes, this seems to be a common issue recently recently. User gets no
feedback that the query failed due to syntax error, and they keep waiting for
the spinning wheel in the UI
xiangfu0 opened a new issue #7345:
URL: https://github.com/apache/pinot/issues/7345
Tried
`bin/quick-start-complex-type-handling-offline.sh`
```
select id, "payload.commits.distinct" from githubEvents limit 10
```
works
but not
```
select id, payload.commits.distinct
ksnijjer opened a new issue #7344:
URL: https://github.com/apache/pinot/issues/7344
Firing a query with syntax error provides no feedback to the end user and UI
continues to show the spinning icon while underlying API has already returned
an error response.
For e.g running something
kishoreg commented on issue #6921:
URL: https://github.com/apache/pinot/issues/6921#issuecomment-902873751
Is Arrow a specification or an implementation? Pinot already has a lot of
techniques available in Arrow. I am thinking if it's a better idea to implement
Arrow spec on top of Pinot
codecov-commenter edited a comment on pull request #7340:
URL: https://github.com/apache/pinot/pull/7340#issuecomment-902336225
#
This is an automated email from the ASF dual-hosted git repository.
snlee 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 52a2e40 Fix the issue with concurrent
snleee merged pull request #7343:
URL: https://github.com/apache/pinot/pull/7343
--
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:
codecov-commenter edited a comment on pull request #7340:
URL: https://github.com/apache/pinot/pull/7340#issuecomment-902336225
#
codecov-commenter edited a comment on pull request #7340:
URL: https://github.com/apache/pinot/pull/7340#issuecomment-902336225
#
codecov-commenter edited a comment on pull request #7340:
URL: https://github.com/apache/pinot/pull/7340#issuecomment-902336225
#
lfeagan commented on issue #6921:
URL: https://github.com/apache/pinot/issues/6921#issuecomment-902841362
Agreed. I am looking for any binary format that provides more efficient
transfer than JSON over HTTP/1.1. Arrow is a good choice. Avro over gRPC would
also do well based on my
amrishlal commented on a change in pull request #7340:
URL: https://github.com/apache/pinot/pull/7340#discussion_r693098977
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -2596,7 +2596,7 @@ public
jtao15 commented on pull request #7343:
URL: https://github.com/apache/pinot/pull/7343#issuecomment-902820699
LGTM
--
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
amrishlal commented on a change in pull request #7174:
URL: https://github.com/apache/pinot/pull/7174#discussion_r691550554
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/periodictask/ControllerPeriodicTask.java
##
@@ -54,14 +55,27 @@
This is an automated email from the ASF dual-hosted git repository.
tingchen 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 612803e Add an optional flag to disable
chenboat merged pull request #7341:
URL: https://github.com/apache/pinot/pull/7341
--
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:
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
codecov-commenter edited a comment on pull request #7342:
URL: https://github.com/apache/pinot/pull/7342#issuecomment-902394157
#
70 matches
Mail list logo
