Skip to content

Minor fixes to four TPC-H queries fixed query syntax#7

Open
dr-riz wants to merge 13 commits intocartershanklin:masterfrom
dr-riz:master
Open

Minor fixes to four TPC-H queries fixed query syntax#7
dr-riz wants to merge 13 commits intocartershanklin:masterfrom
dr-riz:master

Conversation

@dr-riz
Copy link
Copy Markdown

@dr-riz dr-riz commented Aug 14, 2014

Hi Carter,

Four TPC-H queries did not compile due to some minor typos. I have fixed and tested them with Hive over Tez. I can also confirm that the remaining queries compile without a problem.

Please, accept this humble contribution to your efforts. Thank you. :-)

hdfs and others added 12 commits August 12, 2014 09:28
Create run_tpcds_bby.py
Changed default execution engine to tez
corrected typo to commented out configs set tez.queue.name=apps;
Added a dot.  (regex correction to capture 3 digits and 2 digits after the decimal place in Time taken)
@cartershanklin
Copy link
Copy Markdown
Owner

Thanks I took the query syntax fixes but didn't really want the settings updates.

@dr-riz
Copy link
Copy Markdown
Author

dr-riz commented Nov 3, 2014

Thanks. no worries. :-)

Rizwan Mian :-)
BigData Scientist and Consultant
data++, Toronto
647 533 4249
http://www.linkedin.com/in/rizwanmian
http://www.linkedin.com/profile/view?id=23361025&trk=nav_responsive_tab_profile

On Mon, Nov 3, 2014 at 12:15 AM, Carter Shanklin notifications@github.com
wrote:

Thanks I took the query syntax fixes but didn't really want the settings
updates.

Reply to this email directly or view it on GitHub
#7 (comment)
.

corrected script syntax
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

Successfully merging this pull request may close these issues.

3 participants