chenjunjiedada commented on a change in pull request #374: Migrate spark table
to iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r315935720
##########
File path: hive/src/test/java/org/apache/iceberg/hive/TestHiveMetastore.java
##########
@@ -102,7 +102,7 @@ private TServer newThriftServer(TServerSocket socket,
HiveConf conf) throws Exce
.transportFactory(new TTransportFactory())
.protocolFactory(new TBinaryProtocol.Factory())
.minWorkerThreads(3)
- .maxWorkerThreads(5);
+ .maxWorkerThreads(8);
Review comment:
This is used for the unit test. The spark job needs at least three thrift
server threads since we have three partitions in the unit test, the maximum
value of 5 should be enough while the previous unit tests still not release yet
(default release time is 1800s) which cause failed to connect to thrift server.
----------------------------------------------------------------
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, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
