Skip to content
This repository has been archived by the owner on Oct 20, 2018. It is now read-only.

Error Message Parameter Possibly Incorrect #36

Open
kennethmcfarland opened this issue Jan 8, 2018 · 2 comments
Open

Error Message Parameter Possibly Incorrect #36

kennethmcfarland opened this issue Jan 8, 2018 · 2 comments

Comments

@kennethmcfarland
Copy link
Contributor

System.out.println("Text file dir does not exist: " + args[0]);

I'm just eyeballing here, but it looks like the line above should reference args[1] instead of the args[0].

@kennethmcfarland
Copy link
Contributor Author

FluoConfiguration config = new FluoConfiguration(new File(args[0]));
config.setLoaderThreads(20);
config.setLoaderQueueSize(40);
try (FluoClient fluoClient = FluoFactory.newClient(config);
LoaderExecutor le = fluoClient.newLoaderExecutor()) {
File[] files = new File(args[1]).listFiles();
if (files == null) {
System.out.println("Text file dir does not exist: " + args[0]);
} else {

Noticing line 32 where it references args[0] and compare with line 23 shows the conflict. I think its just an index off by one thing, change line 32 to args[1]. I'd welcome a sanity check, I've had the California flu :(

@keith-turner
Copy link
Member

I'd welcome a sanity check, I've had the California flu :(

@kpm1985 I agree that error message does look incorrect. Hope you are feeling better.

keith-turner added a commit that referenced this issue Jan 13, 2018
Fix for #36 - Error Message Uses Incorrect Argument
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants