-
Notifications
You must be signed in to change notification settings - Fork 457
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
ddl on Postgres #96
Labels
Comments
@bobml Thanks for noticing that. Would you be willing to create a pull request to fix the problem? If you don't have the bandwidth I can take it to the workgroup. |
Hi Clair:
I am lower than a novice on github so I have no idea what "create a pull
request" means. If you could take it to the workgroup,. that would be great.
Thanks,
Bob
…On Fri, Aug 4, 2017 at 1:36 PM, clairblacketer ***@***.***> wrote:
@bobml <https://github.com/bobml> Thanks for noticing that. Would you be
willing to create a pull request to fix the problem? If you don't have the
bandwidth I can take it to the workgroup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#96 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AcBXTFKbsKqjqHiL_djbjJ2_JUTgCyggks5sU1aAgaJpZM4Ot-FP>
.
--
*Bob Lanese*
Data Architect; Institute for Computational Biology
6-102A Wolstein Research Building / 2103 Cornell Rd. / Cleveland, OH USA
44106
www.icompbio.org
Administration and Research Informatics; Case Comprehensive Cancer Center
P: 216.368.0252 / M: 216.346.4029
bob.lanese@case.edu
"Carpe Datum"
"I don't know how the world works, like somehow I missed that day in
school" - J. Abumrad
"This is the dumbest idea I've ever heard, and I'm definitely in." K.
Thompson
"Big data is like teenage sex: everyone talks about it, nobody really knows
how to do it, everyone thinks everyone else is doing it, so everyone claims
they are doing it..." - Dan Ariely
www.theperceivedimage.com
Visit us online at *http://cancer.case.edu <http://cancer.case.edu/>*.
Connect with us:
Facebook: *http://www.facebook.com/CaseCCC
<http://www.facebook.com/CaseCCC> *
Twitter: @caseccc <http://twitter.com/caseccc>
*The Case Comprehensive Cancer Center (Case CCC) is a partnership of the
Case Western Reserve University School of Medicine, the Seidman Cancer
Center of University Hospitals Case Medical Center of Cleveland, and the
Taussig Cancer Institute of the Cleveland Clinic. Through the collaborative
research activities of the partner institutions, the Case CCC pursues its
mission of translating research discoveries into improved treatments for
cancer patients throughout the region and nation. *
|
Thanks @bobml, will do! |
Closed
clairblacketer
added a commit
that referenced
this issue
Aug 8, 2017
…Postgresql but similar issues were found in oracle, impala and sql server files.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the note_nip table there is a column named 'offset' which is a reserved word in Postgres. Should be either renamed or quoted.
The text was updated successfully, but these errors were encountered: