https://pinot.apache.org/ logo
Join SlackCommunities
Powered by
Hi All, <http://localhost:9000/v2/brokers/tenants/...
# general
k
Hi All, http://localhost:9000/v2/brokers/tenants/DefaultTenant?state=ONLINE : “_error”: “Permission is denied for access type ‘READ’ to the endpoint “. It’s happening while i’m trying to connect through pinot-jdbc 0.8.0 version client using user pwd.
m
can you share the code showing what you did that resulted in that error?
k
I have found that was a bug in pinot jdbc client, pushed fix changes for merge.
m
ah cool - this is your fix right? https://github.com/apache/pinot/pull/7774/files
can you update the issue/commit message so that it says 'Pass in Headers when creating PinotConnection'
k
Yes, thanks @User
@User: done
m
oh sorry I didn't explain - I meant to update the pull request rather than the GH issue!
I said the wrong thing
so when it's merged in the message on the commit will be easier to understand
k
@User: Done
m
nice one
@User can I also ask you one thing. When I use the JDBC connector I find that I have to manually include org.apache.commons, otherwise I get this type of error:
Copy code
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/commons/lang3/tuple/Pair
	at org.apache.pinot.client.PinotDriver.connect(PinotDriver.java:60)
	at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:677)
	at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:251)
	at Query3.main(Query3.java:16)
Caused by: java.lang.ClassNotFoundException: org.apache.commons.lang3.tuple.Pair
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
	... 4 more
do you have the same thing?
k
yes and i have added :
Copy code
<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-lang3</artifactId>
    <version>3.12.0</version>
</dependency>
else it would fail.
m
ok cool, same. I guess we should include that dependency alongside the connector
so it works out of the box
k
0.8.0 stable release would fail, if auth is enabled and it’s a bug in jdbc connector that i have raised.
@User: commons-lang3 is coming from
Copy code
<dependency>
  <groupId>org.apache.pinot</groupId>
  <artifactId>pinot-spi</artifactId>
  <scope>provided</scope>
</dependency>
in pinot jdbc clinet which is provided in scope thats why lang3 is missing
pinot-clients pom : add
Copy code
<dependency>
  <groupId>org.apache.commons</groupId>
  <artifactId>commons-lang3</artifactId>
</dependency>
m
is it needed by the java client too?
k
just checking else need to add in jdbc only
@User:Anyways its coming from pinot-spi in java clinet also, that means pinot-clients pom with
Copy code
<dependency>
  <groupId>org.apache.commons</groupId>
  <artifactId>commons-lang3</artifactId>
</dependency>
fix the issue.
m
I think @User has done all the code around this
k
Ok
m
have cc'd @User on both the PRs
k
Thanks 😊
k
that's strange. this error should not come. I will check if there's something wrong. The PR looks good to me but I feel like it should not be needed
k
@User : Are you saying for both the PR’s?
k
Ohh I didn't see the first one. I think the commons lang one should not be needed
k
m
@User as it should already be included or nothing from there is used?
k
already included. anyways, I will check it out
m
@User oh interesting. I wonder whether something in the packaging changed then
k
Yeah, I think there were changes to remove parent dependencies in some modules
Open in Slack
PreviousNext
Powered by