search

ngageoint / hootenanny

Hootenanny conflates multiple maps into a single seamless map.
GNU General Public License v3.0
352 stars 74 forks source link

Suppress osm api test error message when later version than postgres 9.1 have been installed #453

Closed bwitham closed 8 years ago

bwitham commented 8 years ago

I recently built a new hoot machine and the postgres 9.1 version installed now shows this warning about the => operator being deprecated. This causes ServiceOsmApiDbTest.sh to fail. We're aware of the deprecation and have other tickets to remove that operation, so this just needs to be suppressed from the test output for the time being.

bwitham commented 8 years ago

Giving up on this for now. afaik, it only affects me, so I'll deal with the test error for now.

Tried grep'ing out the deprecation warnings but am unable to do so without hosing the rest of the test output.

jasonsurratt commented 8 years ago

@bwitham, This is now impacting @mattjdnv migration of the Jenkins tests to vagrant. Can we just have @mingjiesu remove the deprecated operator?

bwitham commented 8 years ago

Sure. I think @treycaldwell may be already assigned that task. I don't think its a small one.

jasonsurratt commented 8 years ago

Ah, is it mostly in Java land? Sorry, this is starting to come back to me.

bwitham commented 8 years ago

I think that change would be completely in the Java code. Brian mentions as part #414. It may not take that long to do actually. As for this task, you may be able to figure out how to suppress the warning message...I tried some stderr redirection and just couldn't get it work without breaking the test.

bwitham commented 8 years ago

Actually, not completely true...there would be some changes in the api db test scripts for #414, obviously, b/c those are what's causing this message to be logged.

brianhatchl commented 8 years ago

This happens on my vagrant ubuntu vm and it's running pg 9.1. Silly fix, but PR here: https://github.com/ngageoint/hootenanny/pull/477

jasonsurratt commented 8 years ago

I see a bunch of uses of => in c++ including in the osmapi. When @mingjiesu is done refactoring the services API code it would probably be good to have her address this.

bwitham commented 8 years ago

issue merry-go-round here :-) The original intent of this one to suppress the log message was fixed with what Brian did yesterday, so will close. Will assign Ming #414, since I think Trey is working on something else right now.

bwitham commented 8 years ago

fixed by @brianhatchl