Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Header lines option does nothing #30

Closed
dalemyers opened this issue Feb 24, 2014 · 6 comments
Closed

Header lines option does nothing #30

dalemyers opened this issue Feb 24, 2014 · 6 comments

Comments

@dalemyers
Copy link

The option for ignoring header lines is there, however it doesn't actually get used at any point. All lines make it into the results.

@harelba
Copy link
Owner

harelba commented Feb 24, 2014

Hi,

Can you please send an example in which it doesn't work?

I've tried the following and it seems to return the correct answers:
harel@harel-laptop:/dev/github/q$ ./q -H 0 "select * from exampledatafile" | wc -l
248
harel@harel-laptop:
/dev/github/q$ ./q -H 1 "select * from exampledatafile" | wc -l
247
harel@harel-laptop:~/dev/github/q$ ./q -H 2 "select * from exampledatafile" | wc -l
246

@dalemyers
Copy link
Author

So I've looked into it further. It doesn't seem to matter the file. What causes it to break is when I use the '-t' flag to use tab as the delimiter.

@harelba
Copy link
Owner

harelba commented Feb 24, 2014

Oh, yes, you're right. It's a bug in the -t parameter. I'll fix that in the next revision. In the mean time, you can just use -d $'\t' instead of -t to get the tab delimition (The need for the $ before the '\t' is some linux quirk that was the reason i created the -t parameter in the first place). If you combine -d and -H they will both work. Only the -t "shortcut" is broken.

@harelba
Copy link
Owner

harelba commented Mar 1, 2014

Beta version providing column name detection through the header will be released later on today.

@harelba
Copy link
Owner

harelba commented Mar 1, 2014

Beta version 1.3.0b released with this fix. Will close when the version moves out of beta.

@harelba
Copy link
Owner

harelba commented Mar 3, 2014

Fixed in 1.3.0. Closing.

@harelba harelba closed this as completed Mar 3, 2014
@cemeyer cemeyer mentioned this issue Mar 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants