I am seeing <my build> and <Jackie's build> also f...
# pinot-dev
a
I am seeing my build and Jackie's build also failing with the same error message in JDK 1.8 Quickstart (JDK 11, 13 Quickstart passes). So this must be a generic build issue rather than something specific to changes in these PRs? Error messages given below. Any idea what has changed here?
Copy code
[{"message":"BrokerResourceMissingError","errorCode":410}]
...
Executing command: StopProcess -controller -server -broker -zooKeeper
18885
+ true
18886
+ kill -0 3750
18887
.github/workflows/scripts/.pinot_quickstart.sh: line 71: kill: (3750) - No such process
18888
+ break
18889
+ rm -rf /tmp/PinotAdmin/zkData
18890
+ '[' 0 -eq 0 ']'
18891
Batch Quickstart failed: Cannot get correct result for count star query.
18892
+ echo 'Batch Quickstart failed: Cannot get correct result for count star query.'
18893
+ exit 1
18894
Error: Process completed with exit code 1.
m
Seems the quickstart test has become brittle recently. Some JDK version keeps failing randomly and passes on rerun
I suspect enabling the lead controller resources by default adds some extra workload on the controller, which causes the issue
Let's see if extending the setup time fixes the issue: https://github.com/apache/incubator-pinot/pull/6746
m
It was approved, so I took the liberty to merge the PR.
@Jackie ^^
j
Was planning to run tests more times lol. Let's see if the problem gone in master
n
doesnt look like it’s helping
j
Yeah, same issue..
m
Argh
x
seems this time it’s unit test failure
pinot-spark 😛
j
a
Pinot Quickstart on JDK 1.8
is still failing. I am wondering what are the options here? Should we consider removing this check? Is someone still using JDK1.8?
n
but its not just 1.8
it is randomly failing for all of the quickstart with the differnet jdk versions
a
I think the
BrokerResourceMissingError
exception means that Broker doesn't know about the
baseballStats
table. Exception is thrown in
BrokerResponseNative.java
.
Maybe this step is failing (looking at logs), but I don't see any error messages associated with this:
Copy code
Executing command: AddTable -tableConfigFile /tmp/1617896415131/baseballStats_1617896415254.config -schemaFile /tmp/1617896375079/baseballStats/baseballStats_schema.json -controllerProtocol http -controllerHost fv-az201-716 -controllerPort 9000 -user null *** -exec
j
@Amrish Lal can you try to rebase your code with latest master branch? I think jackie has merged some PR to increase the setup time in the test
n
i dont think that is helping.. i rebased and still see the same issue
j
The issue is still there.. Basically the controller rest endpoint is not responding, thus the table is not created. I'm trying to dig into it
We can ignore the quick-start tests for now. Please make sure the unit tests and integration tests pass before merging