Dang. Sorry Nick wasting your time. I thought I'd purged the binaries (purged the 'bin' stuff.. not 'pkg').
Thanks for the feedback. RC sunk. Let me put up another addressing all issues you've raised. Good one, St.Ack On Fri, Jan 9, 2015 at 11:39 AM, Nick Dimiduk <[email protected]> wrote: > [+1] verify signature and checksums > [-0] src release appears to contain binaries. This was intentional? > > $ tar tzf htrace-3.1.0-incubating-src.tar.gz | grep '\.a' > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > github.com/alecthomas/units.a > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > github.com/gorilla/context.a > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > github.com/gorilla/mux.a > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > github.com/jmhodges/levigo.a > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > gopkg.in/alecthomas/kingpin.v1.a > > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/org/apache/htrace/common.a > > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/org/apache/htrace/conf.a > > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/org/apache/htrace/resource.a > > htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/org/apache/htrace/test.a > > [-1] build with `mvn clean package` > > Build fails with the following output. Should I have go-1.3 installed > instead? > > [INFO] --- maven-antrun-plugin:1.6:run (go_compile) @ htrace-core --- > [WARNING] Parameter tasks is deprecated, use target instead > [INFO] Executing tasks > > main: > [exec] Failed to run 'git rev-parse HEAD' > [exec] 2015/01/09 11:29:51 Bundled > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/web as > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/web/index.html.go > [exec] 2015/01/09 11:29:51 Bundled > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/web as > > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/web/nested__nested.html.go > [exec] fatal: Not a git repository (or any of the parent directories): > .git > [exec] org/apache/htrace/bundler > [exec] org/apache/htrace/common > [exec] org/apache/htrace/conf > [exec] org/apache/htrace/resource > [exec] org/apache/htrace/test > [exec] org/apache/htrace/htrace > [exec] org/apache/htrace/htraced > [exec] # org/apache/htrace/htrace > [exec] src/org/apache/htrace/htrace/cmd.go:26: import > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > gopkg.in/alecthomas/kingpin.v1.a: object is [darwin amd64 go1.3.3 > X:precisestack] expected [darwin amd6$ > go1.4 X:precisestack] > [exec] # org/apache/htrace/htraced > [exec] src/org/apache/htrace/htraced/datastore.go:25: import > /private/tmp/htrace-3.1.0-incubating/htrace-core/src/go/pkg/darwin_amd64/ > github.com/jmhodges/levigo.a: object is [darwin amd64 go1.3.3 > X:precisestack] expected [darwin a$ > d64 go1.4 X:precisestack] > > My platform is: > > $ go version > go version go1.4 darwin/amd64 > $ brew info leveldb > > > leveldb: stable 1.18 (bottled) > https://github.com/google/leveldb/ > /usr/local/Cellar/leveldb/1.18 (24 files, 1.3M) * > Poured from bottle > From: > https://github.com/Homebrew/homebrew/blob/master/Library/Formula/leveldb.rb > ==> Dependencies > Required: snappy ✔ > $ mvn --version > Apache Maven 3.2.3 (33f8c3e1027c3ddde99d3cdebad2656a31e8fdf4; > 2014-08-11T13:58:10-07:00) > Maven home: /usr/local/Cellar/maven/3.2.3/libexec > Java version: 1.7.0_67, vendor: Oracle Corporation > Java home: > /Library/Java/JavaVirtualMachines/jdk1.7.0_67.jdk/Contents/Home/jre > Default locale: en_US, platform encoding: UTF-8 > OS name: "mac os x", version: "10.9.5", arch: "x86_64", family: "mac" > > > On Fri, Jan 9, 2015 at 11:27 AM, Nick Dimiduk <[email protected]> wrote: > > > I have another nit: sine this is a source release, it would be good to > > include a BUILDING.txt file in the root of the tgz that has instructions, > > dependencies, &c. It's fine to have this on release notes on the site, > but > > it's a little strange for the distribution to not be self-contained. > > > > On Fri, Jan 9, 2015 at 11:19 AM, Stack <[email protected]> wrote: > > > >> Agree that migration doc is not a blocker on this RC. > >> St.Ack > >> > >> On Fri, Jan 9, 2015 at 10:57 AM, Jake Farrell <[email protected]> > >> wrote: > >> > >> > I do not see these issues as blocking, but we probably should add a > >> > CHANGELOG and update it each release along with adding javadocs. We > can > >> > create tickets and address these issues for the next release > >> > > >> > -Jake > >> > > >> > On Fri, Jan 9, 2015 at 12:44 PM, Lewis John Mcgibbney < > >> > [email protected]> wrote: > >> > > >> > > If we can clarify if the issues are NOT blocking this RC progressing > >> > then I > >> > > am more than happy to vreview folks. > >> > > Please let me know, I was in the process of reviewing until this > email > >> > came > >> > > in... and quite right too! Good catches and excellent points. > >> > > Thanks > >> > > lewis > >> > > > >> > > On Thu, Jan 8, 2015 at 6:29 PM, Nick Dimiduk <[email protected]> > >> wrote: > >> > > > >> > > > I'm trying out this RC over in HBase code. We've broken API > >> > compatibility > >> > > > in at least two commits: > >> > > > > >> > > > HTRACE-1 changes the SpanReceiver interface. HBase was > instantiating > >> > > > SpanReceivers itself and calling .configure(HTraceConfiguration) > on > >> > each > >> > > > one. Looks like now the expectation is that SpanReceiver > >> > implementations > >> > > > provide a constructor that takes a single parameter of > >> > > HTraceConfiguration. > >> > > > > >> > > > HTRACE-16 refactors the TraceTree interface a good bit. The handy > >> > > > getRoots() method has been replaced with the less obvious > >> > > > getSpansByParent().find(Span.ROOT_SPAN_ID) and > >> .getSpansByParentIdMap() > >> > > is > >> > > > also an invocation of getSpansByParent().find(). > >> > > > > >> > > > Neither of these changes are a big deal, but none of this code has > >> > > > javadocs, so it's not obvious how the contract changed. I had to > >> look > >> > at > >> > > > diffs and tests to decipher the new usage. This means that a > simple > >> > > > search/replace is not sufficient for existing users to upgrade. > >> > > > > >> > > > For reference, my patch is over on HBASE-12810. > >> > > > > >> > > > I don't think this is enough to sink the RC, but my understanding > of > >> > the > >> > > > earlier discussion was that this release would be a simple > >> > search/replace > >> > > > kind of upgrade. We need to document these idiom changes for > folks. > >> > > > > >> > > > -n > >> > > > > >> > > > On Thu, Jan 8, 2015 at 5:06 PM, Stack <[email protected]> wrote: > >> > > > > >> > > > > I've posted our sixth htrace-3.1.0 RC here: > >> > > > > > >> > > > > http://people.apache.org/~stack/htrace-3.1.0-incubatingRC5/ > >> > > > > > >> > > > > Maven artifacts are here: > >> > > > > > >> > > > > > >> > > > > >> > > https://repository.apache.org/content/repositories/orgapachehtrace-1007 > >> > > > > > >> > > > > The main changes from RC5 are packaging fixups and licensing > >> > > attribution. > >> > > > > > >> > > > > The release is a src-only tarball for now. Later we can add > binary > >> > > > bundles > >> > > > > to our release after we have better sense of what we as a > project > >> > would > >> > > > > like to deliver. > >> > > > > > >> > > > > This release is mainly a change of packaging from org.htrace to > >> > > > > org.apache.htrace to suit our new home here in Apache Incubator > >> but > >> > it > >> > > > does > >> > > > > also includes 34 resolved issues [1] including the beginnings of > >> an > >> > > > htrace > >> > > > > daemon whose intent is to make it so there is a low barrier > >> > collecting > >> > > > > cluster traces as well as a new flume receiver. > >> > > > > > >> > > > > Beware that Apache Incubator, org.apache.htrace is not > compatible > >> > with > >> > > > its > >> > > > > former self, org.htrace: the package name has changed but so has > >> the > >> > > JSON > >> > > > > serialization format. > >> > > > > > >> > > > > Shall we make this release candidate our first incubator > release? > >> > > > > > >> > > > > Lets keep the vote period short (We'll have to run another vote > >> over > >> > in > >> > > > > incubator general after this one if I understand the process > >> > properly). > >> > > > > > >> > > > > Please vote +1/0/-1 by Monday, January 9th, 2015. > >> > > > > > >> > > > > Thanks, > >> > > > > St.Ack > >> > > > > > >> > > > > 1. > >> > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > https://issues.apache.org/jira/issues/?jql=project%20%3D%20HTRACE%20AND%20status%20%3D%20resolved%20AND%20fixVersion%20%3D%203.1.0%20ORDER%20BY%20issuetype%20DESC > >> > > > > > >> > > > > >> > > > >> > > > >> > > > >> > > -- > >> > > *Lewis* > >> > > > >> > > >> > > > > >
