I think just the settings you need. On Tue, Sep 9, 2014 at 12:52 PM, Chris Rohr <[email protected]> wrote:
> Thanks Aaron. Would we need to set all of the config or just the frame > size? > > On Tue, Sep 9, 2014 at 12:40 PM, Aaron McCurry <[email protected]> wrote: > > Yes you are probably correct. To set it on the client side you will need > > to add a blur-site.properties file into your classpath that has the > > framesize set correctly. We should probably add an issue to have the > > clients figure out what they should be set to by negotiating with the > > server. > > > > Aaron > > > > On Tue, Sep 9, 2014 at 11:52 AM, Chris Rohr <[email protected]> > wrote: > > > >> Hi all, > >> > >> We are having an issue with hitting the max frame size on thrift. I > >> had gotten some guidance to use the config option > >> blur.thrift.max.frame.size to increase from the 16M default, however, > >> this doesn't seem to affect the error. > >> > >> I have a theory as to why we are still getting the error and wanted to > >> run it by everyone. I think this error is being thrown on the client > >> side connecting to the controllers. I see that the BlurClient sets up > >> the BlurClientManager which in turn creates a ClientPool. The > >> ClientPool seems to set the max frame size but the code will always > >> hit the default 16M and there doesn't seem to be a way to adjust that > >> value from the client code. > >> > >> Here is the code from the ClientPool I was looking at: > >> > >> static { > >> try { > >> BlurConfiguration config = new BlurConfiguration(); > >> _idleTimeBeforeClosingClient = > >> config.getLong(BLUR_CLIENTPOOL_CLIENT_CLOSE_THRESHOLD, > >> TimeUnit.SECONDS.toNanos(30)); > >> _clientPoolCleanFrequency = > >> config.getLong(BLUR_CLIENTPOOL_CLIENT_CLEAN_FREQUENCY, > >> TimeUnit.SECONDS.toMillis(3)); > >> _maxFrameSize = config.getInt(BLUR_THRIFT_MAX_FRAME_SIZE, 16384000); > >> } catch (Exception e) { > >> throw new RuntimeException(e); > >> } > >> checkAndRemoveStaleClients(); > >> } > >> > >> BlurClient and BlurClientManager do not have any setters or references > >> to affect this code in the ClientPool. > >> > >> Could this be the issue we are seeing? > >> > >> Thanks, > >> Chris > >> >
