From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [hail patch 1/1] Make host, url, orig_path dynamic Date: Tue, 20 Jul 2010 16:34:19 -0400 Message-ID: <4C46084B.8010104@garzik.org> References: <20100720141645.3dc01964@lembas.zaitcev.lan> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:sender:message-id:date:from :user-agent:mime-version:to:cc:subject:references:in-reply-to :content-type:content-transfer-encoding; bh=eryktAVoLebVtmsFszqwkIYZOm3CpDT+uaUA3OOtae4=; b=Q9myU8fx6I5XKSLzfAQOVYVAv4jR5SWkfkyl0AtMS8bV7aPTLRYoEVWG5ZtIqgA9ec yUVeZfWoq0Tw3bs/GzHjeMmk5eoRA8HBXF4K+zL/aQHXgizWwZaGPRpVu16qOnT/TGIG WwEQ1IXntmyYww4wkE/QzgLO/USqDyoLimawE= In-Reply-To: <20100720141645.3dc01964@lembas.zaitcev.lan> Sender: hail-devel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Pete Zaitcev Cc: Project Hail List On 07/20/2010 04:16 PM, Pete Zaitcev wrote: > Some of my performance tests for tabled hit truncation again: > > [zaitcev@hitlain tests]$ ./poke5 -v -h niphredil.zaitcev.lan -u auser > -p apass -b test -o -k testkey-hitlain/73b84a11e6d83c65e45853338d646042 > -f testdir/73b84a11e6d83c65e45853338d646042 > * About to connect() to niphredil.zaitcev.lan port 80 (#0) > * Trying fec0::1:219:b9ff:fe58:7ad6... * TCP_NODELAY set > * connected > * Connected to niphredil.zaitcev.lan (fec0::1:219:b9ff:fe59:7ad6) port 80 (#0) > > PUT /test/testkey-hitlain/73b84a11e6d83c65e45853338d HTTP/1.1 > Accept: */* > Host: niphredil.zaitcev.lan > Date: Tue, 20 Jul 2010 01:07:33 +0000 > Authorization: AWS testuser:RefcbVYgr2m9KTRxOrCfr4zzfPE= > Content-Length: 214745088 > Expect: 100-continue > > * The requested URL returned error: 403 > > As you can see, the path in PUT is truncated, and this causes 403 > since it's included into a hash. > > The patch addresses this issue and a bunch of other fixed-size > strings before we hit that. > > Signed-off-by: Pete Zaitcev > > --- > lib/hstor.c | 147 +++++++++++++++++++++++++++++++++++--------------- > 1 file changed, 104 insertions(+), 43 deletions(-) applied