Thanks!

On Tue, Feb 25, 2014 at 7:55 PM, Tim Williams <[email protected]> wrote:

> On Tue, Feb 25, 2014 at 7:49 PM, Aaron McCurry <[email protected]> wrote:
> > @@ -116,7 +117,7 @@ public class HdfsFieldManager extends
> BaseFieldManager {
> >        List<String> fieldNames = new ArrayList<String>();
> >        for (FileStatus fileStatus : listStatus) {
> >          if (!fileStatus.isDir()) {
> > -          fieldNames.add(fileStatus.getPath().getName());
> > +
>  fieldNames.add(fileStatus.getPath().getName().replace(TYPE_FILE_EXT,
> > ""));
> >          }
> >        }
> >
> > Do you think that this will cause problems for fields that are <some
> > family>.type<any suffix> ?  Maybe we should check that the file ends with
> > the TYPE_FILE_EXT and substring up to the beginning of the suffix?
>
> Ahh, literally have 'type' as the prefix of their column... yeah,
> that'd cause problems, nice catch.  I'll get a test for that one...
>
> --tim
>

Reply via email to