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 3028055 Fix quick-start tests (#6751)
Jackie-Jiang merged pull request #6751:
URL: https://github.com/apache/incubator-pinot/pull/6751
--
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.
For queries about this se
malbx opened a new pull request #6765:
URL: https://github.com/apache/incubator-pinot/pull/6765
## Description
Fix the exception thrown in the case that a specified table name does not
exist (#6328). As opposed to returning `BROKER_RESOURCE_MISSING_ERROR`, which
can be a bit misleading
fx19880617 closed pull request #6762:
URL: https://github.com/apache/incubator-pinot/pull/6762
--
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.
For queries about this serv
Jackie-Jiang closed pull request #6764:
URL: https://github.com/apache/incubator-pinot/pull/6764
--
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.
For queries about this se
Jackie-Jiang commented on pull request #6764:
URL: https://github.com/apache/incubator-pinot/pull/6764#issuecomment-816429661
Close the test PR
--
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 th
This is an automated email from the ASF dual-hosted git repository.
nehapawar 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 409e5da SegmentUploader impl (#6740
npawar merged pull request #6740:
URL: https://github.com/apache/incubator-pinot/pull/6740
--
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.
For queries about this service,
npawar commented on pull request #6740:
URL: https://github.com/apache/incubator-pinot/pull/6740#issuecomment-816328306
Merging this since the unit/integration tests have passed, and the
quickstart failure is a known issue that @Jackie-Jiang is working on. Across
several attempts the quick
npawar merged pull request #6743:
URL: https://github.com/apache/incubator-pinot/pull/6743
--
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.
For queries about this service,
This is an automated email from the ASF dual-hosted git repository.
nehapawar pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 4fd84be Added guava cache package to common relocated packages (#6758)
add 5815f4c Fix hadoop-mapr
fx19880617 commented on a change in pull request #6762:
URL: https://github.com/apache/incubator-pinot/pull/6762#discussion_r610188601
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/QuickstartRunner.java
##
@@ -204,11 +209,30 @@ public void create
fx19880617 commented on a change in pull request #6762:
URL: https://github.com/apache/incubator-pinot/pull/6762#discussion_r610186872
##
File path: .github/workflows/pinot_tests.yml
##
@@ -85,4 +85,6 @@ jobs:
with:
java-version: ${{ matrix.java }}
-
Jackie-Jiang commented on a change in pull request #6762:
URL: https://github.com/apache/incubator-pinot/pull/6762#discussion_r610169601
##
File path: .github/workflows/pinot_tests.yml
##
@@ -85,4 +85,6 @@ jobs:
with:
java-version: ${{ matrix.java }}
npawar commented on a change in pull request #6740:
URL: https://github.com/apache/incubator-pinot/pull/6740#discussion_r610161942
##
File path:
pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/src/main/java/org/apache/pinot/plugin/segmentuploader/SegmentUploaderDef
npawar commented on a change in pull request #6740:
URL: https://github.com/apache/incubator-pinot/pull/6740#discussion_r610161478
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/util/IngestionUtils.java
##
@@ -185,38 +177,107 @@ public static String buildSegment(
npawar commented on a change in pull request #6740:
URL: https://github.com/apache/incubator-pinot/pull/6740#discussion_r610161239
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/util/IngestionUtils.java
##
@@ -185,38 +177,107 @@ public static String buildSegment(
codecov-io commented on pull request #6763:
URL: https://github.com/apache/incubator-pinot/pull/6763#issuecomment-816270147
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6763?src=pr&el=h1)
Report
> Merging
[#6763](https://codecov.io/gh/apache/incubator-pinot/pull/6763?s
Author: xiangfu
Date: Thu Apr 8 21:12:53 2021
New Revision: 46956
Log:
Delete release-0.4.0-rc1
Removed:
dev/incubator/pinot/apache-pinot-incubating-0.4.0-rc1/
-
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
Jackie-Jiang opened a new pull request #6763:
URL: https://github.com/apache/incubator-pinot/pull/6763
## Description
In RealtimeJsonIndexQuickStart, use transform function to generate the json
string instead of modifying the kafka stream
Misc fixes:
- Support ingesting `"null"
fx19880617 opened a new pull request #6762:
URL: https://github.com/apache/incubator-pinot/pull/6762
## Description
Add a description of your PR here.
A good description should include pointers to an issue or design document,
etc.
## Upgrade Notes
Does this PR prevent a zero dow
mayankshriv opened a new pull request #6761:
URL: https://github.com/apache/incubator-pinot/pull/6761
- Added pinot-segment-local module that contains all the segment related
implementations for the existing local Pinot segment.
- Modules that pinot-segment-local depends on:
fredster33 opened a new pull request #6760:
URL: https://github.com/apache/incubator-pinot/pull/6760
## Description
Grammar edits, commented out some sections of the pull request template so
that even if users do not delete that portion of the PR, it does not appear in
the final submitt
Jackie-Jiang merged pull request #6758:
URL: https://github.com/apache/incubator-pinot/pull/6758
--
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.
For queries about this se
Jackie-Jiang closed issue #6757:
URL: https://github.com/apache/incubator-pinot/issues/6757
--
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.
For queries about this service
fx19880617 opened a new pull request #6759:
URL: https://github.com/apache/incubator-pinot/pull/6759
## Description
Fix the issue that PinotFs doesn't init before the first get in
SegmentGenerationJobRunner
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume
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 4fd84be Added guava cache package to c
daniellavoie commented on a change in pull request #6758:
URL: https://github.com/apache/incubator-pinot/pull/6758#discussion_r609919617
##
File path: pinot-common/pom.xml
##
@@ -312,6 +312,10 @@
com.google.common.base
shaded.com.g
fx19880617 commented on a change in pull request #6758:
URL: https://github.com/apache/incubator-pinot/pull/6758#discussion_r609917933
##
File path: pinot-common/pom.xml
##
@@ -312,6 +312,10 @@
com.google.common.base
shaded.com.goo
fx19880617 commented on a change in pull request #6758:
URL: https://github.com/apache/incubator-pinot/pull/6758#discussion_r609917933
##
File path: pinot-common/pom.xml
##
@@ -312,6 +312,10 @@
com.google.common.base
shaded.com.goo
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 25bd754 Add more logging for segment
chenboat merged pull request #6754:
URL: https://github.com/apache/incubator-pinot/pull/6754
--
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.
For queries about this servic
daniellavoie opened a new pull request #6758:
URL: https://github.com/apache/incubator-pinot/pull/6758
## Description
Our internal CI and local builds would encounter a classpath issue on
`pinot-hadoop` test phase.
```
java.lang.NoSuchMethodError:
com.google.common.cache.
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815906675
Fixed it by adding the guava `cache` package to the `pinot-common` shade
relocation configuration. PR coming through.
--
This is an automated message from the Apache
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815895589
I added `-verbose` to the surefire plugin and found the following
interesting trace
```
org.apache.maven.plugins
maven-surefire
daniellavoie removed a comment on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815840218
Found this:
```
$ MAVEN_OPTS="-verbose" mvn package| grep
com.google.common.cache.CacheBuilder
[Loaded com.google.common.cache.CacheBuilder from
fi
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815840218
Found this:
```
$ MAVEN_OPTS="-verbose" mvn package| grep
com.google.common.cache.CacheBuilder
[Loaded com.google.common.cache.CacheBuilder from
file:/Appl
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815832546
Honestly, I'm clueless, hadoop-client 2.7.0 uses method signatures that
matches guava 20
--
This is an automated message from the Apache Git Service.
To respond to th
alexandervivas closed issue #6433:
URL: https://github.com/apache/incubator-pinot/issues/6433
--
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.
For queries about this servi
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815816132
Might be caused by the shade plugin configuration.
```
maven-shade-plugin
3.2.1
daniellavoie commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815814309
Seems to run fine from the IDE, but still getting the error with `mvn test`
```
pinot-hadoop % mvn dependency:tree -Dverbose -Dincludes=com.google.guava
[INF
Jackie-Jiang commented on issue #6752:
URL:
https://github.com/apache/incubator-pinot/issues/6752#issuecomment-815534588
Json index only supports exact match. In order to do text match, you need to
use `jsonExtractScalar`, e.g. `select ... where jsonExtractScalar(json_col,
'$.message', 'S
Jackie-Jiang commented on issue #6757:
URL:
https://github.com/apache/incubator-pinot/issues/6757#issuecomment-815524841
I cannot reproduce this locally. Based on the error message, seems the
failure is caused guava library version conflict. Can you please check if there
are multiple vers
43 matches
Mail list logo
