You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bigtop.apache.org by "Evans Ye (JIRA)" <ji...@apache.org> on 2019/02/09 14:05:00 UTC
[jira] [Comment Edited] (BIGTOP-2012) Add new tests for webhdfs
[ https://issues.apache.org/jira/browse/BIGTOP-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16764150#comment-16764150 ]
Evans Ye edited comment on BIGTOP-2012 at 2/9/19 2:04 PM:
----------------------------------------------------------
OK. I've tested this patch and fixes several bugs. The refined patch is uploaded.
The original patch has 3 tests that are failing. I'll explain with lines in your patch:
* testMkdir
line 106-107 the check for the string is always false because the result is separated into lines:
{code}
{"FileStatuses":{"FileStatus":[
]}}
{code}
I remove the check since a 200 response should be enough.
* testCreateFile
Line 126 define the output string to check is "HTTP/1.1 307 Temporary Redirect". However the resulting sting is "HTTP/1.1 307 TEMPORARY_REDIRECT".
* testGetFileChecksum
The REST call is redirected to datanodes, hence -L option is required for curl to continue the call and get desired response, which is the checksum of a file.
[~kollavenkat] whould you please review and comment?
was (Author: evans_ye):
OK. I've tested this patch and fixes several bugs. The refined patch is uploaded.
The original patch has 3 tests that are failing. I'll explain with lines in your patch:
* testMkdir
line 106-107 the check for the string is always false because the result is separated into lines:
{code}
{"FileStatuses":{"FileStatus":[
]}}
{code}
I remove the check since a 200 response should be enough.
* testCreateFile
Line 126 define the output string to check is "HTTP/1.1 307 Temporary Redirect". However the resulting sting is "HTTP/1.1 307 TEMPORARY_REDIRECT".
* testGetFileChecksum
The REST call is redirected to datanodes, hence -L option is required for curl to continue the call and get desired response, which is the checksum of a file.
> Add new tests for webhdfs
> -------------------------
>
> Key: BIGTOP-2012
> URL: https://issues.apache.org/jira/browse/BIGTOP-2012
> Project: Bigtop
> Issue Type: Sub-task
> Components: general
> Affects Versions: 1.0.0
> Reporter: Venkata Srinivasa Rao Kolla
> Assignee: Venkata Srinivasa Rao Kolla
> Priority: Minor
> Fix For: backlog
>
> Attachments: BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch, BIGTOP-2012.patch
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)