shahsank3t commented on pull request #6148:
URL: https://github.com/apache/incubator-pinot/pull/6148#issuecomment-712691245
@npawar @mayankshriv
made the changes so we no longer need to press "Enter", "Space" or "Comma"
to make a chip before saving tags.
This is an automated email from the ASF dual-hosted git repository.
mcvsubbu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new cf675a5 Setup classpath correctly
Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-713044724
@jackjlli I see the problem, basically `tyrus` copied the code from
`grizzly` and put it in one package. If you upgrade `tyrus` to `1.15` to be
compatible with
mcvsubbu merged pull request #6160:
URL: https://github.com/apache/incubator-pinot/pull/6160
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
lgo closed issue #6145:
URL: https://github.com/apache/incubator-pinot/issues/6145
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
lgo commented on issue #6145:
URL:
https://github.com/apache/incubator-pinot/issues/6145#issuecomment-712979402
Egh sorry, this was a bit of noise. I completely mixed up issues we were
seeing -- instead we were seeing expected behaviour (broker pruning and
immedately returning zero
yupeng9 commented on a change in pull request #6153:
URL: https://github.com/apache/incubator-pinot/pull/6153#discussion_r508779940
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/util/TableConfigUtils.java
##
@@ -265,6 +265,12 @@ protected static void
Jackie-Jiang commented on a change in pull request #6153:
URL: https://github.com/apache/incubator-pinot/pull/6153#discussion_r508712115
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/util/TableConfigUtils.java
##
@@ -265,6 +265,12 @@ protected static void
Jackie-Jiang commented on a change in pull request #6153:
URL: https://github.com/apache/incubator-pinot/pull/6153#discussion_r508712115
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/util/TableConfigUtils.java
##
@@ -265,6 +265,12 @@ protected static void
fx19880617 opened a new issue #6161:
URL: https://github.com/apache/incubator-pinot/issues/6161
Provide ACL policy for the s3 bucket write.
This is an automated message from the Apache Git Service.
To respond to the message,
pradeepgv42 commented on issue #6161:
URL:
https://github.com/apache/incubator-pinot/issues/6161#issuecomment-713080879
I believe, in S3PinotFs, acls need to be set in put/copy object requests
such that bucket owner is given full access to the objects created. Something
like below, most
snleee merged pull request #6166:
URL: https://github.com/apache/incubator-pinot/pull/6166
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.
snlee pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 2484f5b add option flags for controller host:port, jar path, and
admin path (#6163)
add df4911d Bump
Author: jlli
Date: Wed Oct 21 05:37:23 2020
New Revision: 42005
Log:
Update Pinot KEYS file for
Modified:
release/incubator/pinot/KEYS
Modified: release/incubator/pinot/KEYS
==
--- release/incubator/pinot/KEYS
fx19880617 commented on issue #6146:
URL:
https://github.com/apache/incubator-pinot/issues/6146#issuecomment-713304807
An improvement for this: https://github.com/apache/incubator-pinot/pull/6165
This will limit the idealstates update parallelism to at most the number of
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a commit to branch data-generator-options-parser
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit c5d63770009211264a3fbcdbae1d93d786cbaa83
Author: Alexander Pucher
AuthorDate: Tue Oct
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch data-generator-options-parser
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at c5d6377 add option flags for controller host:port, jar path, and
admin path
This
apucher opened a new pull request #6163:
URL: https://github.com/apache/incubator-pinot/pull/6163
## Description
generator.sh: Adds optional flags for controller host:port, jar path, and
pinot-admin path. This enables the script to be used from non-controller
containers, or outside of
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 08c46a8 Support using ordinals in GROUP BY and ORDER BY clause (#6152)
add 2484f5b add option flags
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 08c46a8 Support using ordinals in GROUP BY and ORDER BY clause (#6152)
add 2484f5b add option flags
codecov-io edited a comment on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-713175399
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6164?src=pr=h1)
Report
> Merging
yupeng9 commented on a change in pull request #6167:
URL: https://github.com/apache/incubator-pinot/pull/6167#discussion_r508991010
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/upsert/PartitionUpsertMetadataManager.java
##
@@ -29,97 +30,142 @@
/**
*
jackjlli merged pull request #6162:
URL: https://github.com/apache/incubator-pinot/pull/6162
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.
jlli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from da451a8 Add more validation for upsert config (#6153)
add 81248d1 Update tyrus version (#6162)
No new
jackjlli commented on a change in pull request #6162:
URL: https://github.com/apache/incubator-pinot/pull/6162#discussion_r508860231
##
File path: pinot-tools/pom.xml
##
@@ -110,6 +110,10 @@
testng
test
+
Review comment:
That's good to have.
mcvsubbu commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r508867945
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -1641,23 +1643,30 @@
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 81248d1 Update tyrus version (#6162)
add 08c46a8 Support using ordinals in GROUP BY and ORDER BY
codecov-io commented on pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#issuecomment-713177186
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6165?src=pr=h1)
Report
> Merging
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 9ba9f4e Add table level lock for segment upload
add 2d95abd Add table
codecov-io edited a comment on pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#issuecomment-713177186
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6165?src=pr=h1)
Report
> Merging
snleee opened a new pull request #6166:
URL: https://github.com/apache/incubator-pinot/pull/6166
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new da451a8 Add more validation for
Jackie-Jiang merged pull request #6153:
URL: https://github.com/apache/incubator-pinot/pull/6153
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit f3a94cc976c4a63509f0844db14b0cf5c95e46fd
Author: Xiang Fu
AuthorDate: Tue Oct 20
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at f3a94cc Add table level lock for segment upload
This branch includes the
kbastani opened a new pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164
## Description
Added a GIF animated video of navigating the features of the Pinot 0.5.0
user interface (cluster manager, query console, and ZK browser).
fx19880617 opened a new pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165
## Description
Add table level lock for segment upload to avoid unnecessary race conditions
to cause idealstates update failure.
fx19880617 merged pull request #6152:
URL: https://github.com/apache/incubator-pinot/pull/6152
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
apucher merged pull request #6163:
URL: https://github.com/apache/incubator-pinot/pull/6163
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
codecov-io commented on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-713175399
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6164?src=pr=h1)
Report
> Merging
codecov-io edited a comment on pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#issuecomment-713177186
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6165?src=pr=h1)
Report
> Merging
jackjlli opened a new pull request #6162:
URL: https://github.com/apache/incubator-pinot/pull/6162
## Description
This PR bumps up Tyrus version to 1.15. This is to solve the issue that
pinot-tools modules happens to use two classes with the same name and package.
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch update-tyrus-version
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 2a82a45 Update tyrus version
This branch includes the following new commits:
new
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch update-tyrus-version
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 2a82a45bbfbc43c488166c9079b7f5f6feaaf629
Author: Jack Li(Analytics Engineering)
AuthorDate: Tue
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard f3a94cc Add table level lock for segment upload
add 9ba9f4e Add table
jackjlli commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-713134195
@Jackie-Jiang Thanks for looking into it. I've opened a new PR to bump up
the Tyrus version to 1.15
(https://github.com/apache/incubator-pinot/pull/6162). Once that's
codecov-io commented on pull request #6162:
URL: https://github.com/apache/incubator-pinot/pull/6162#issuecomment-713151690
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6162?src=pr=h1)
Report
> Merging
fx19880617 closed issue #6155:
URL: https://github.com/apache/incubator-pinot/issues/6155
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
apucher commented on pull request #6163:
URL: https://github.com/apache/incubator-pinot/pull/6163#issuecomment-713174458
I'll upgrade this eventually. this is mainly about making headway towards
benchmarking
This is an
jackjlli closed pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139
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
codecov-io edited a comment on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-713175399
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6164?src=pr=h1)
Report
> Merging
jackjlli commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-713180695
Closing this PR since https://github.com/apache/incubator-pinot/pull/6162
has been merged.
This is an
codecov-io edited a comment on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-713175399
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6164?src=pr=h1)
Report
> Merging
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 2d95abd Add table level lock for segment upload
add 8a02f09 Add table
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 8a02f09 Add table level lock for segment upload
add 145a314 Add table
codecov-io edited a comment on pull request #6153:
URL: https://github.com/apache/incubator-pinot/pull/6153#issuecomment-75276
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6153?src=pr=h1)
Report
> Merging
Jackie-Jiang commented on a change in pull request #6162:
URL: https://github.com/apache/incubator-pinot/pull/6162#discussion_r508857156
##
File path: pinot-tools/pom.xml
##
@@ -110,6 +110,10 @@
testng
test
+
Review comment:
I don't think you
codecov-io edited a comment on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-713175399
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6164?src=pr=h1)
Report
> Merging
fx19880617 commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r508901950
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -1641,23 +1643,30 @@
Jackie-Jiang opened a new pull request #6167:
URL: https://github.com/apache/incubator-pinot/pull/6167
## Description
Part of a series of PRs for #4261
Re-implement the `PartitionUpsertMetadataManager` to correctly handle the
following 2 scenarios:
1. Reload the segment which
npawar commented on a change in pull request #6124:
URL: https://github.com/apache/incubator-pinot/pull/6124#discussion_r508945680
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/minion/MinionTaskMetadataUtils.java
##
@@ -0,0 +1,81 @@
+/**
+ * Licensed to the
mcvsubbu commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r508948376
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -1641,23 +1643,30 @@
codecov-io commented on pull request #6167:
URL: https://github.com/apache/incubator-pinot/pull/6167#issuecomment-713253167
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6167?src=pr=h1)
Report
> Merging
codecov-io edited a comment on pull request #6167:
URL: https://github.com/apache/incubator-pinot/pull/6167#issuecomment-713253167
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6167?src=pr=h1)
Report
> Merging
mayankshriv commented on pull request #6166:
URL: https://github.com/apache/incubator-pinot/pull/6166#issuecomment-713278645
Would be good to document what's new in the release and why we need it.
This is an automated
fx19880617 commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r508990839
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -1641,23 +1643,30 @@
66 matches
Mail list logo