KKcorps merged PR #9802:
URL: https://github.com/apache/pinot/pull/9802
--
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.apache
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 42354206b7 Add memory optimized dimension ta
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038094950
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/BaseMultipleSegmentsConversionExecutor.java:
##
@@ -242,
jayeshchoudhary commented on PR #9844:
URL: https://github.com/apache/pinot/pull/9844#issuecomment-1335238495
@jadami10 can you add a sorting example with GB or some other unit as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038164529
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/MinionTaskUtils.java:
##
@@ -67,7 +72,35 @@ static Pinot
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038164688
##
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/MergeRollupMinionClusterIntegrationTest.java:
##
@@ -79,15 +81,18 @@ public class MergeRollupM
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038165234
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/BaseMultipleSegmentsConversionExecutor.java:
##
@@ -276,
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038165869
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/BaseMultipleSegmentsConversionExecutor.java:
##
@@ -276,
KKcorps commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038166185
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/BaseMultipleSegmentsConversionExecutor.java:
##
@@ -276,
jadami10 commented on PR #9844:
URL: https://github.com/apache/pinot/pull/9844#issuecomment-1335392698
I changed the code locally to return random bytes. Sorting seems to work
well.
```
function getRandomInt(min, max) {
min = Math.ceil(min);
max = Math.floor(m
walterddr opened a new pull request, #9902:
URL: https://github.com/apache/pinot/pull/9902
this fixes: #9695.
--
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
walterddr commented on code in PR #9850:
URL: https://github.com/apache/pinot/pull/9850#discussion_r1038292605
##
pinot-query-runtime/src/test/resources/queries/Comparisons.json:
##
@@ -390,6 +390,7 @@
"queries": [
{ "sql": "SELECT val BETWEEN small AND big FROM {tbl
walterddr commented on code in PR #9850:
URL: https://github.com/apache/pinot/pull/9850#discussion_r1038292605
##
pinot-query-runtime/src/test/resources/queries/Comparisons.json:
##
@@ -390,6 +390,7 @@
"queries": [
{ "sql": "SELECT val BETWEEN small AND big FROM {tbl
walterddr commented on code in PR #9850:
URL: https://github.com/apache/pinot/pull/9850#discussion_r1038292605
##
pinot-query-runtime/src/test/resources/queries/Comparisons.json:
##
@@ -390,6 +390,7 @@
"queries": [
{ "sql": "SELECT val BETWEEN small AND big FROM {tbl
walterddr commented on code in PR #9850:
URL: https://github.com/apache/pinot/pull/9850#discussion_r1038296545
##
pinot-query-runtime/src/test/resources/queries/MathFuncs.json:
##
@@ -0,0 +1,439 @@
+{
+ "addition": {
+"tables": {
+ "numTbl": {
+"schema": [
+
walterddr commented on code in PR #9850:
URL: https://github.com/apache/pinot/pull/9850#discussion_r1038297108
##
pinot-query-runtime/src/test/resources/queries/MathFuncs.json:
##
@@ -0,0 +1,439 @@
+{
+ "addition": {
+"tables": {
+ "numTbl": {
+"schema": [
+
walterddr commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038367972
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -97,14 +97,26 @@ public static String substr(String input, int beginIn
walterddr commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038369710
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -119,6 +131,17 @@ public static String concat(String input1, String in
zhtaoxiang commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038378843
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/BaseMultipleSegmentsConversionExecutor.java:
##
@@ -2
zhtaoxiang commented on code in PR #9825:
URL: https://github.com/apache/pinot/pull/9825#discussion_r1038386220
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/MinionTaskUtils.java:
##
@@ -67,7 +73,40 @@ static Pi
agavra commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038415383
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -119,6 +131,17 @@ public static String concat(String input1, String input
walterddr commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038422529
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -119,6 +131,17 @@ public static String concat(String input1, String in
Jackie-Jiang commented on PR #9875:
URL: https://github.com/apache/pinot/pull/9875#issuecomment-1335672091
@vvivekiyer Adding a test in `LLCRealtimeClusterIntegrationTest` should be
relatively easy. You can pick a column with dictionary and a column without
dictionary, then switch the encod
Jackie-Jiang merged PR #9844:
URL: https://github.com/apache/pinot/pull/9844
--
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.a
Jackie-Jiang closed issue #9843: In Pinot dashboard, sorting on table size
takes alphabetical order
URL: https://github.com/apache/pinot/issues/9843
--
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 t
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 9b8809fea5 add table sort function for table size
Jackie-Jiang merged PR #9896:
URL: https://github.com/apache/pinot/pull/9896
--
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.a
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 9b8809fea5 add table sort function for table size (#9844)
add 85e413ebe8 In Kafka consumer, seek offset only when
walterddr commented on code in PR #9826:
URL: https://github.com/apache/pinot/pull/9826#discussion_r1038436355
##
pinot-query-runtime/src/test/resources/queries/WithStatements.json:
##
@@ -0,0 +1,81 @@
+{
+ "with_statement_tests": {
+"psql": "7.8.1",
+"tables": {
+
agavra commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038440166
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -97,14 +97,26 @@ public static String substr(String input, int beginIndex
Jackie-Jiang commented on code in PR #9902:
URL: https://github.com/apache/pinot/pull/9902#discussion_r1038441723
##
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/Connection.java:
##
@@ -165,7 +163,33 @@ public Future executeAsync(String query)
@Deprec
walterddr commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038442543
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -97,14 +97,26 @@ public static String substr(String input, int beginIn
codecov-commenter commented on PR #9902:
URL: https://github.com/apache/pinot/pull/9902#issuecomment-1335747379
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9902?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
61yao opened a new pull request, #9903:
URL: https://github.com/apache/pinot/pull/9903
Instructions:
1. The PR has to be tagged with at least one of the following labels (*):
1. `feature`
2. `bugfix`
3. `performance`
4. `ui`
5. `backward-incompat`
6. `re
61yao commented on PR #9895:
URL: https://github.com/apache/pinot/pull/9895#issuecomment-1335825931
> > > WDYT about implementing this as a rewriter that just swaps the left
and right tables?
> >
> >
> > That's a good idea. but then we need to deal the situation where we
output t
agavra opened a new issue, #9904:
URL: https://github.com/apache/pinot/issues/9904
## Scenario A
https://user-images.githubusercontent.com/3172405/205389930-f76dfdfc-391f-4713-9ed8-8e73778a9db4.png";>
## Scenario B
https://user-images.githubusercontent.com/3172405/2053900
agavra commented on issue #9904:
URL: https://github.com/apache/pinot/issues/9904#issuecomment-1335863433
I see the following in the logs:
```
2022/12/02 13:25:40.861 ERROR [MailboxSendOperator]
[query_worker_on_8842_port-1-thread-8] Exception occur while sending data via
mailbox
agavra commented on issue #9904:
URL: https://github.com/apache/pinot/issues/9904#issuecomment-1335871113
nevermind - this happens because MV columns aren't supported
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
agavra closed issue #9904: [multistage] table aliasing doesn't work with SELECT
*
URL: https://github.com/apache/pinot/issues/9904
--
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 commen
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-segment-upload-race-condition
in repository https://gitbox.apache.org/repos/asf/pinot.git
at f8f6a9fdfe Fix race condition when 2 segment upload occurred for the
same segment
This branc
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-segment-upload-race-condition
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit f8f6a9fdfe09686730e2409c20a631dda5256a79
Author: Jack Li(Analytics Engineering)
AuthorDate: Fri
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-segment-upload-race-condition
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard f8f6a9fdfe Fix race condition when 2 segment upload occurred for the
same segment
new d5
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-segment-upload-race-condition
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit d5f494503ce0a9988a7664c04fa5c52b30b6d9d8
Author: Jack Li(Analytics Engineering)
AuthorDate: Fri
jackjlli opened a new pull request, #9905:
URL: https://github.com/apache/pinot/pull/9905
There is a race condition when it took too much time for the 1st segment
upload to process (due to slow PinotFS access), which leads to the 2nd attempt
of segment upload, and the 2nd segment upload suc
agavra commented on PR #9895:
URL: https://github.com/apache/pinot/pull/9895#issuecomment-1335927107
@61yao instead of an optimization rule it should be implemented as a
rewriter. I tested this and it works:
```java
public class RightJoinRewriter extends SqlShuttle {
publi
abhs50 commented on issue #9788:
URL: https://github.com/apache/pinot/issues/9788#issuecomment-1335933999
Apologies for delay. @Jackie-Jiang : Is there a config which controls this :
https://github.com/apache/pinot/blob/dcc496f63b4488989808ec144a7e2ffcc4697c75/pinot-core/src/main/java/org/ap
abhs50 commented on issue #9788:
URL: https://github.com/apache/pinot/issues/9788#issuecomment-1335934766
Additionally is there way to keep Pinot Server in Error logging mode instead
of `info` by default. We are using helm installation way of deploying Pinot.
--
This is an automated messa
codecov-commenter commented on PR #9905:
URL: https://github.com/apache/pinot/pull/9905#issuecomment-1335939785
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9905?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
zhtaoxiang opened a new pull request, #9906:
URL: https://github.com/apache/pinot/pull/9906
Previously, the splitBlock logic throws `Runtime failure:
java.lang.ArithmeticException: / by zero`. This PR fixes this issue.
--
This is an automated message from the Apache Git Service.
To re
codecov-commenter commented on PR #9906:
URL: https://github.com/apache/pinot/pull/9906#issuecomment-1335966281
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9906?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
61yao commented on PR #9895:
URL: https://github.com/apache/pinot/pull/9895#issuecomment-1335978224
> ```java
> SqlShuttle
> ```
I see. Let me try!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
walterddr merged PR #9902:
URL: https://github.com/apache/pinot/pull/9902
--
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
walterddr closed issue #9695: [bug] ConnectorFactory cannot get livebroker
mapping
URL: https://github.com/apache/pinot/issues/9695
--
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 comme
This is an automated email from the ASF dual-hosted git repository.
rongr 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 4ae6d35ff1 [hotfix] broker selection not using tab
codecov-commenter commented on PR #9892:
URL: https://github.com/apache/pinot/pull/9892#issuecomment-1336082626
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9892?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
joshigaurava opened a new pull request, #76:
URL: https://github.com/apache/pinot-site/pull/76
https://user-images.githubusercontent.com/68965367/205429689-05471e61-ec71-4bf8-b176-d7cd8b585e7d.png";>
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
kishoreg commented on PR #76:
URL: https://github.com/apache/pinot-site/pull/76#issuecomment-1336103058
We should point at the issue and blog of join feature coming soon and what's
available in alphas beta GA
--
This is an automated message from the Apache Git Service.
To respond to the m
57 matches
Mail list logo
