[
https://issues.apache.org/jira/browse/BIGTOP-1342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14055790#comment-14055790
]
jay vyas commented on BIGTOP-1342:
----------------------------------
Okay ! ... Functionally, it worked out of the box, 92% pass, 7% fail like in
BIGTOP-1341.
Now, there are some significant whitespace complaints, etc... when applying the
patch.
[~mbukatov] Do you want to try and fix those whitespace errors and comment on
the errors which are unfixable? ( I see errors for examplein patch lines 84,
547, 583, 605,638) when i run "git am BIGTOP-1342.1.patch" on Master.
The way i do formatting of groovy code is here :
https://issues.apache.org/jira/browse/BIGTOP-1240?focusedCommentId=13920396&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13920396.
Anyways... once those are fixed, as far as i can tell, we can push this guy in
--- just in time for the 0.8.0 release, it ill be cool to announce that this
release includes major HCFS improvements in the tests.
> Make TestCLI usable for both HDFS and HCFS
> ------------------------------------------
>
> Key: BIGTOP-1342
> URL: https://issues.apache.org/jira/browse/BIGTOP-1342
> Project: Bigtop
> Issue Type: Improvement
> Components: Tests
> Affects Versions: 0.8.0
> Reporter: Martin Bukatovic
> Attachments: BIGTOP-1342.1.patch
>
>
> Current TestCLI test cases are currently only runnable on HDFS. Since the
> most
> test cases are applicable on any hadoop filesystem, it makes sense to
> make it general in hcfs sense.
> While most test cases are hcfs generic, some cases are only applicable to
> HDFS,
> so I propose split the current code into:
> * general HCFS superclass (with most cases in {{testHCFSConf.xml}} file)
> * HDFS specific subclass (with hdfs only cases in {{testHDFSConf.xml}})
> I would like to keep {{testHCFSConf.xml}} as a common base for any hadoop
> filesystem, which would require introduction of several additional variables
> to
> catch minor differences between GlusterFS and HDFS. This should be good enough
> for other hcfs implementations as well, but I didn't tested it.
> Before proposing patch for this, it make sense to have the following resolved:
> * BIGTOP-1341 TestCLI cleanup
> * BIGTOP-1334 Add DFS support to TestCLI
--
This message was sent by Atlassian JIRA
(v6.2#6252)
