Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/895
@parthchandra can you please review this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/905
@paul-rogers @jinfengni can you please review this one?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1199
Nice work, @kkhatua! This seems very useful!
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1184
@parthchandra can you please review it. @jiang-wu Parth is traveling and he
would be able to review it next week.
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1204
@ppadma can you review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1204
@arina-ielchiieva or @vdiravka can you review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1207
There is a large number of files affected - it might be good to get this in
early so that we don't have to keep rebasing. @vrozov/ @Ben-Zvi can you glance
through the changes?
@Be
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1207
@arina-ielchiieva or @Ben-Zvi would it make sense to merge this in earlier
before the batch commits later in the week? the number of files changes are
large so don't want @ilooner to have to r
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/920
@Ben-Zvi can you please review for it to be included in the current sprint?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/930
@vvysotskyi is it possible to change the port number for Lilith to avoid
the conflict?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/967
@parthchandra can you review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/961
@paul-rogers please review.
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/981
@paul-rogers could you please review this one?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/990
@amansinha100 or @arina-ielchiieva can you please review this?
---
GitHub user priteshm opened a pull request:
https://github.com/apache/drill/pull/994
Merge from latest
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/priteshm/drill master
Alternatively you can review and apply these changes
Github user priteshm closed the pull request at:
https://github.com/apache/drill/pull/994
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/949
@paul-rogers, @kkhatua can you provide some more information on the test
case that failed? Hopefully, @dprofeta can replicate it in his environment.
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/978
@sohami can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/976
@sachouche can you please take a final look? If it looks good, maybe one of
the committers can include this for the 1.12 release. @arina-ielchiieva ?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1030
@ppadma can you review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1014
@ilooner can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1025
@amansinha100 can you review this change?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/976
@dprofeta will you be able to address the issues before the release?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1028
@parthchandra can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1038
@parthchandra can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1040
@arina-ielchiieva can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1041
@parthchandra or @adityakishore since you'll have made changes to these
files before, can you review the changes here?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1001
@ppadma @paul-rogers I see that @sachouche addressed the comments in the
JIRA - is this one ready to merge?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1047
@sachouche can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1071
@paul-rogers can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1001
@sachouche can you update this PR?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1086
@paul-rogers, @parthchandra can you review/ comment on this change?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1072
@paul-rogers Is this ready for commit?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1045
@arina-ielchiieva - any chance this can be committed with the batch commits
tomorrow? otherwise, @ilooner would need to rebase again!
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1106
@amansinha100 can you please review it?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/
@vrozov can you please review this change?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1011
@sachouche @vrozov @arina-ielchiieva please review
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1119
@arina-ielchiieva is on vacation. @vrozov, @Ben-Zvi can you take a look?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1101
@Ben-Zvi can you please do a final review?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1105
@arina-ielchiieva is this bug ready to commit?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1133
@parthchandra or @arina-ielchiieva can you review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1096
@gparai did you get a chance to address @amansinha100 's comment for this
PR?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1101
Spoke with Chun, he will run the tests and update the PR with the test
results.
---
Github user priteshm commented on a diff in the pull request:
https://github.com/apache/drill/pull/1135#discussion_r171731905
--- Diff: distribution/src/resources/drillbit.sh ---
@@ -45,7 +45,7 @@
# configuration file. The option takes precedence over the
# DRILL_CONF_DIR
Github user priteshm commented on a diff in the pull request:
https://github.com/apache/drill/pull/1135#discussion_r172619940
--- Diff: distribution/src/resources/drillbit.sh ---
@@ -45,7 +45,7 @@
# configuration file. The option takes precedence over the
# DRILL_CONF_DIR
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
@chunhui-shi or @HanumathRao can you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1166
@parthchandra would you please review this?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1158
@vdiravka please do link the JIRA that you will create.
@vrozov can you take a final look?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1173
@parthchandra can you please review this change?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
@HanumathRao, @chunhui-shi any more comments from you?
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
Thanks, @chunhui-shi - marked it as ready-to-commit since the original
feature was already merged to 1.13. The batch committer this week can take
another look as well.
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1179
@bitblender can you please review this?
---
52 matches
Mail list logo