Dave, Michael, I totally agree with you. But dropping a column in a databse script without making sure that the data in the column is preserved is bad. Seriously. I don't care about news-aggregator. We don't use it and I got this to work at a client site after upgrade. My only point is: Don't drop database columns *unless* you provide an upgrade script which stores the column value somewhere else.
Michael, can you answer me the question why it is mandatory that the package_id is dropped from na_sources? It obviously will break *any* .LRN upgrade that is using news-aggregator. Whereas if we just keep the package_id, news-aggregator would still run.
