Running into an issue using latest pinot-presto im...
# troubleshootinga
Ashish
04/21/2021, 9:29 PMRunning into an issue using latest pinot-presto image against pinot latest.
presto:default> select * from baseballstats;
Query 20210421_202103_00011_isaaq, FAILED, 1 node
Splits: 17 total, 0 done (0.00%)
1:00 [0 rows, 0B] [0 rows/s, 0B/s]
Query 20210421_202103_00011_isaaq failed: null value in entry: Server_172.19.0.2_7000=null
Ashish
04/21/2021, 9:30 PM2021-04-21T203607.566Z ERROR nioEventLoopGroup-7-1 org.apache.pinot.$internal.org.apache.pinot.core.transport.DataTableHandler Caught exception while deserializing data table of size: 966 from server: 172.19.0.2_O
java.lang.UnsupportedOperationException: Unsupported data table version: 3
at org.apache.pinot.$internal.org.apache.pinot.core.common.datatable.DataTableFactory.getDataTable(DataTableFactory.java:37)
at org.apache.pinot.$internal.org.apache.pinot.core.transport.DataTableHandler.channelRead0(DataTableHandler.java:67)
at org.apache.pinot.$internal.org.apache.pinot.core.transport.DataTableHandler.channelRead0(DataTableHandler.java:36)
at org.apache.pinot.$internal.io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
at org.apache.pinot.$internal.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:374)
Ashish
04/21/2021, 9:31 PMIs there a backward incompatible change introduced recently? Do I need to recreate pinot-presto image again with latest pinot client library?
j
Jackie
04/21/2021, 9:58 PM@Xiang Fu I think we need to pick up the latest pinot code in the connector
Jackie
04/21/2021, 9:59 PMWe recently upgraded the data table version: https://github.com/apache/incubator-pinot/pull/6710
x
Xiang Fu
04/21/2021, 10:00 PM
i see
Xiang Fu
04/21/2021, 10:00 PM
i need to make this change then
Xiang Fu
04/21/2021, 10:00 PM
hmm
Xiang Fu
04/21/2021, 10:01 PM
is it possible to specify the version from server request?
Xiang Fu
04/21/2021, 10:03 PM
I think this change is not yet released. So from presto side, I have no reference to it ?
Xiang Fu
04/21/2021, 10:03 PM
or it’s already in 0.7.1?
j
Jackie
04/21/2021, 10:04 PMNo, it's not released yet. There is a server config
pinot.server.instance.currentDataTableVersionJackie
04/21/2021, 10:05 PMIf set to 2, it should remain the same behavior
x
Xiang Fu
04/21/2021, 10:05 PM
I see. @Ashish can you give it a try by adding
pinot.server.instance.currentDataTableVersion=2Xiang Fu
04/21/2021, 10:05 PM
hmm
j
Jackie
04/21/2021, 10:08 PM@Xiang Fu We should probably pairing the pinot-presto image with the pinot image, e.g. for each pinot release have a pinot-presto release?
x
Xiang Fu
04/21/2021, 10:09 PM
it’s too much overhead for managing 😛 I will try
Xiang Fu
04/21/2021, 10:10 PM
just if this feature not in 0.7.1 release, then we should wait for it’s in 0.8.0
Xiang Fu
04/21/2021, 10:10 PM
then remove
j
Jackie
04/21/2021, 10:11 PMTrue. The issue is caused by running pinot-presto image for
0.7.1x
Xiang Fu
04/21/2021, 10:11 PM
yeah
Xiang Fu
04/21/2021, 10:12 PM
Maybe we can add this to default helm pinot server config
Xiang Fu
04/21/2021, 10:12 PM
so new users won’t experience failure
j
Jackie
04/21/2021, 10:14 PMYeah, that works. We can remove that after the next release
x
Xiang Fu
04/21/2021, 10:14 PM
cool
Xiang Fu
04/21/2021, 10:18 PM
a
Ashish
04/21/2021, 10:43 PMTrying out the workaround now