Return-Path: 
 <avro-dev-return-2934-apmail-hadoop-avro-dev-archive=hadoop.apache.org@hadoop.apache.org>
Delivered-To: apmail-hadoop-avro-dev-archive@minotaur.apache.org
Received: (qmail 18563 invoked from network); 2 Feb 2010 07:22:50 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3)
  by minotaur.apache.org with SMTP; 2 Feb 2010 07:22:50 -0000
Received: (qmail 13206 invoked by uid 500); 2 Feb 2010 07:22:50 -0000
Delivered-To: apmail-hadoop-avro-dev-archive@hadoop.apache.org
Received: (qmail 13115 invoked by uid 500); 2 Feb 2010 07:22:49 -0000
Mailing-List: contact avro-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:avro-dev-help@hadoop.apache.org>
List-Unsubscribe: <mailto:avro-dev-unsubscribe@hadoop.apache.org>
List-Post: <mailto:avro-dev@hadoop.apache.org>
List-Id: <avro-dev.hadoop.apache.org>
Reply-To: avro-dev@hadoop.apache.org
Delivered-To: mailing list avro-dev@hadoop.apache.org
Received: (qmail 13105 invoked by uid 99); 2 Feb 2010 07:22:49 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Feb 2010 07:22:49 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=10.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Feb 2010 07:22:40 +0000
Received: from brutus.apache.org (localhost [127.0.0.1])
	by brutus.apache.org (Postfix) with ESMTP id B99AD234C1EF
	for <avro-dev@hadoop.apache.org>; Mon,  1 Feb 2010 23:22:18 -0800 (PST)
Message-ID: <1496509951.21341265095338759.JavaMail.jira@brutus.apache.org>
Date: Tue, 2 Feb 2010 07:22:18 +0000 (UTC)
From: "Philip Zeyliger (JIRA)" <jira@apache.org>
To: avro-dev@hadoop.apache.org
Subject: [jira] Commented: (AVRO-321) Run Java RPC interop tests again
In-Reply-To: <874243390.222311263418314465.JavaMail.jira@brutus.apache.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394
X-Virus-Checked: Checked by ClamAV on apache.org


    [ https://issues.apache.org/jira/browse/AVRO-321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12828508#action_12828508 ] 

Philip Zeyliger commented on AVRO-321:
--------------------------------------

Looks good overall.  The proof will be when we add a language or two to this suite.  Also, should there be an ant target that runs at least what we have here?  Or, rather, a build.sh target?

For my edification/education, what's the difference between '@' and '$' in ant, in the segment below?
{noformat}
        <sysproperty key="test.dir" value="@{test.dir}"/>
        <sysproperty key="share.dir" value="${share.dir}"/>
{noformat}

bq. RpcReceiveTool.java--sleep(1000)

Why did you need to insert a sleep in here?

bq. Usage: uri protocolFile message_name [ -data d | -file f ]"

I think this might be more appropriately "uri protocol_file message_name (--data d|--file f)" to indicate that one of data or file is required.  We should file a bug for the fact that HttpServer doesn't let you specify what IP to bind to.  (Same goes for RpcSendTool.) 

We should in theory check the scheme of the URI too for valid values.

bq. datum = Util.datumFromFile(file.value(opts)); 

Do we need to check that the file's schema and the protocol schema are identical?  Or, rather, read the file using the protocol schema: yes, I think we need to do the latter.

bq. static Object datumFromFile(String file)

I'm a bit of a worrywart: if this sends only one request (it's conceivable for it to replay a set of them), we should perhaps check that in.hasNext() is false.

bq. TestRpcReceiveAndSendTools.java

There's no test for the "-file" path.

bq. share/test/interop/bin/test_rpc_interop.sh: sleep 1

Might be nice to leave a comment why that sleep needs to be there.  I tried a couple of things, but couldn't get around it.

> Run Java RPC interop tests again
> --------------------------------
>
>                 Key: AVRO-321
>                 URL: https://issues.apache.org/jira/browse/AVRO-321
>             Project: Avro
>          Issue Type: Test
>          Components: build, java
>            Reporter: Jeff Hammerbacher
>            Assignee: Doug Cutting
>             Fix For: 1.3.0
>
>         Attachments: AVRO-321.patch, AVRO-321.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.