A week or so I added dev-support/build-proto.sh, a little script to build protobuf stuff.
https://issues.apache.org/jira/browse/HBASE-7490 I've been modifying protobuf a bit recently -- when you update run the script, protoc seems to generate exactly the same content unless you change proto source files. I think the argument for not having this run by default in the build is having it would require protoc installed on your machine to even build HBase. This also assumes all versions of protoc continues to generate the same code from different version (thrift's compiler was really annoying because every version generated code with a different api in every version). Jon. On Sat, Jan 12, 2013 at 3:38 PM, lars hofhansl <la...@apache.org> wrote: > I see. Is there a way to only regenerate when they changed (ala make or ant)? > We do that sort of thing at Salesforce (not for protobufs, but other similar > problems), that way you see what generated files your change affected and you > can still check them in. > > -- Lars > > > > ________________________________ > From: Matteo Bertozzi <theo.berto...@gmail.com> > To: dev@hbase.apache.org; lars hofhansl <la...@apache.org> > Sent: Saturday, January 12, 2013 3:33 PM > Subject: Re: ProtoBuf generation? > > > I think that the idea was, the .proto files are not changing much overtime > so, we can commit the generated code, and avoid having the protobuf compiler > installed. > if we regenerated the protos everytime (mvn ...) we require protoc to be > installed. > > > Matteo > > On Sat, Jan 12, 2013 at 3:27 PM, lars hofhansl <la...@apache.org> wrote: > > Is there a reason why this in not integrated in the maven build? >>If any of the .proto files changed shouldn't it automatically rebuild the >>generated classes? >> >> >>-- Lars >> >> >> >>________________________________ >> From: Matteo Bertozzi <theo.berto...@gmail.com> >>To: dev@hbase.apache.org >>Sent: Saturday, January 12, 2013 3:14 PM >>Subject: Re: ProtoBuf generation? >> >> >>There's a README inside the hbase-protocol/src/main/protobuf directory >>https://github.com/apache/hbase/blob/trunk/hbase-protocol/src/main/protobuf/README.txt >> >>On Sat, Jan 12, 2013 at 3:12 PM, Jean-Marc Spaggiari < >>jean-m...@spaggiari.org> wrote: >> >>> Thanks for the prompt reply. >>> >>> Should we add that into the HBase book where we explain how to build? >>> Might be helpful? >>> >>> JM >>> >>> 2013/1/12, Ted Yu <yuzhih...@gmail.com>: >>> > Under hbase-protocol module, run: >>> > >>> > protoc -Isrc/main/protobuf --java_out=src/main/java >>> > <path-to-dor-proto-file> >>> > >>> > Cheers >>> > >>> > On Sat, Jan 12, 2013 at 2:28 PM, Jean-Marc Spaggiari < >>> > jean-m...@spaggiari.org> wrote: >>> > >>> >> Hi, >>> >> >>> >> I have update the Client.Proto file and would like to re-generate the >>> >> classes. >>> >> >>> >> I tried mvn clean package -DskipTests and mvn clean install >>> >> -DskipTests but the file is still not generated. >>> >> >>> >> Which command should I run to re-generate it? >>> >> >>> >> Thanks, >>> >> >>> >> JM >>> >> >>> > >>> -- // Jonathan Hsieh (shay) // Software Engineer, Cloudera // j...@cloudera.com
